Skip to content

skpkg: set up scaffold and moved over docs, src, tests, news #165

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

Merged
merged 3 commits into from
Jun 22, 2025

Conversation

kwagoner94
Copy link

No description provided.

Copy link
Contributor

@sbillinge sbillinge left a comment

Choose a reason for hiding this comment

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

this is a great start! In general, pay close attention to things that appear as deleted in these modified files. We often want to keep things from the old package that are being deleted. OK if you miss some, that is what I am here for, and feel free to ask questions.

@@ -6,22 +6,20 @@ build-backend = "setuptools.build_meta"
name = "diffpy.srmise"
dynamic=['version', 'dependencies']
authors = [
{ name="Simon J.L. Billinge group", email="simon.billinge@gmail.com" },
{name="Luke Granlund", email="granlund@pa.msu.edu"},
Copy link
Contributor

Choose a reason for hiding this comment

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

we want to keep Luke's name. We never want to "drop" authors.

pyproject.toml Outdated
readme = "README.rst"
requires-python = ">=3.11, <3.14"
classifiers = [
'Development Status :: 3 - Alpha',
'Development Status :: 5 - Production/Stable',
Copy link
Contributor

Choose a reason for hiding this comment

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

this should probably stay as 3 or 4

pyproject.toml Outdated
@@ -45,9 +42,6 @@ template = "{tag}"
dev_template = "{tag}"
dirty_template = "{tag}"

[project.scripts]
Copy link
Contributor

Choose a reason for hiding this comment

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

we need this.

@kwagoner94
Copy link
Author

Ok seems like I missed those. Will keep an eye out in the future

@sbillinge
Copy link
Contributor

They are honestly easy to catch by looking at the PR on github....

@kwagoner94
Copy link
Author

Ok I see. I will double check the files changed before making the PR in the future

@kwagoner94
Copy link
Author

In a case like this, should I go back and fix the files that were modified?

@kwagoner94 kwagoner94 closed this Jun 22, 2025
@kwagoner94 kwagoner94 reopened this Jun 22, 2025
@kwagoner94
Copy link
Author

I have fixed the file

@sbillinge sbillinge merged commit 451da4d into diffpy:migration Jun 22, 2025
1 check passed
@sbillinge
Copy link
Contributor

nice @kwagoner94 perfectly done. sync your main to upstream and let's go to the next step.

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