Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes on naming and small improvements #30

Merged
merged 8 commits into from
Aug 26, 2024

Conversation

JosePizarro3
Copy link
Contributor

Closes #26 and #28

@blueraft can you give it a look? Ideally, this should be merged asap, I just did some changes on naming and stuff.

Btw, if I add example_uploads, testing does not work because of not being able to import the ExampleUploadsEntryPoint class. But it is not relevant for these changes.

@JosePizarro3
Copy link
Contributor Author

JosePizarro3 commented Aug 23, 2024

@lauri-codes do you mind checking on this? Ahmed is on holidays and I need this for the Hackathon

(I can also fix the documentation regarding plugins)

Copy link
Collaborator

@lauri-codes lauri-codes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here are some comments.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Change newapp for app_entry_point

Fix typo readme
@JosePizarro3
Copy link
Contributor Author

Here are some comments.

Ok, done. I changed back to schema_package(s).

@JosePizarro3
Copy link
Contributor Author

@lauri-codes , you want to give another look or can I merge? I tested it, and all works: https://github.com/JosePizarro3/example-plugin/tree/pyscf-new-template

@lauri-codes
Copy link
Collaborator

@JosePizarro3: Looks good, go ahead and merge

@JosePizarro3
Copy link
Contributor Author

Ah, I cannot merge. I don't have write access.

@lauri-codes lauri-codes merged commit 236339a into FAIRmat-NFDI:main Aug 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feedback on autogenerating new plugins
2 participants