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

Added demokit metadata #76

Closed
wants to merge 2 commits into from
Closed

Conversation

ianmkenney
Copy link
Member

Adding demokit to the registry.

It is a Python package that implements the RMSF analysis class found in MDAnalysis core.

@ianmkenney
Copy link
Member Author

@MDAnalysis/mdakits-reviewers this is ready for review!

Copy link
Member

@orbeckst orbeckst left a comment

Choose a reason for hiding this comment

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

Do you need this to actually show up in the registry?

@@ -0,0 +1,82 @@
# TEMPLATE MDAKit file
Copy link
Member

Choose a reason for hiding this comment

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

replace with something else so that it doesn't look like a template

Comment on lines +4 to +10
# Please replace ALL entries with appropriate content for YOUR MDAKit.
# Below we use the placeholder GH_HOST_ACCOUNT for the GitHub account where
# the source code repository is held, typically your username or the
# organization that you're part off.
# MYPROJECT is the name of your project (the repository name and here
# we assume that this is also the PyPi/conda package name) whereas
# MYPACKAGE is how you import it in python.
Copy link
Member

Choose a reason for hiding this comment

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

I'd remove this, too.

## str: the organisation name the MDAKit falls under
## str: the development status of the MDAKit
## See https://pypi.org/classifiers/ for development status classifiers.
development_status: Productions/Stable
Copy link
Member

Choose a reason for hiding this comment

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

bold statement :-)

@ianmkenney
Copy link
Member Author

@orbeckst We don't want to merge it, it's for the video

@ianmkenney ianmkenney closed this Sep 21, 2023
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.

2 participants