Skip to content

Conversation

kwagoner94
Copy link
Contributor

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.

dynamic=['version', 'dependencies']
authors = [
{ name="Simon J.L. Billinge group", email="[email protected]" },
{name="Luke Granlund", email="[email protected]"},
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
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

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
Contributor 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
Contributor Author

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

@kwagoner94
Copy link
Contributor 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
Contributor 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.

sbillinge added a commit that referenced this pull request Jul 9, 2025
* skpkg: set up scaffold and moved over docs, src, tests, news (#165)

* fix: chagned line length to 115

* chore: move LICENSE files over to scaffold

* fix: Fixed issues with pyproject.toml

* chore: move necessary files over to scaffold (#166)

* fix: chagned line length to 115

* chore: move LICENSE files over to scaffold

* chore: github workflows, codecovs, codespell

* chore: maintenance on CHANGLOG CODE_OF_CONDUCT etc

* chore: move neccessary files over, clean up ones that arent needed

* skpkg: fix pyproject.toml

* [pre-commit.ci] auto fixes from pre-commit hooks

* chore: Replace scikit-package-logo-text.png and example-table.rst with placeholder files, fix docformatter for arbitrary.py

* [pre-commit.ci] auto fixes from pre-commit hooks

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* chore: move over rest of doc and src files (#168)

* chore: move over rest of doc and src files

* chore: run pre-commit

* chore: fix index.rst back to original

* Rename doc → docs, update references, and fix test.txt (#171)

---------

Co-authored-by: kwagoner94 <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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