Skip to content

Modernize repository configuration - #190

Merged
yguclu merged 17 commits into
masterfrom
repo_maintenance
Oct 1, 2026
Merged

yguclu merged 17 commits into
masterfrom
repo_maintenance

Conversation

@FrederikSchnack

@FrederikSchnack FrederikSchnack commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This is the first PR in a sequence of general improvements.

Modernize the repository configuration and testing infrastructure without changing SymPDE’s production code.

Changes

  • Replace the legacy combined CI workflow with a dedicated testing workflow.
  • Run CI when pull requests are opened, reopened, synchronized, or marked ready for review.
  • Modernize project and build metadata in pyproject.toml.
  • Define optional dependency groups for testing and documentation.
  • Move pytest and coverage configuration into pyproject.toml.
  • Update installation and testing instructions in the README.
  • Move TODO.rst to old/ – see Look into old/ #195.
  • Change README to md.
  • Add AUTHORS file.
  • Add CHANGELOG.md.

Delete the unmaintained old/ snapshots and bin/run_tests.sh. The script references test modules that no longer exist, so it advertises a broken workflow while the deleted snapshots duplicate history already preserved by Git.
Move pytest configuration and development dependencies into pyproject.toml, test installed packages across supported Python versions, and collect coverage as a CI artifact.

Keep the tested SymPy upper bound in place while #106 tracks the compatibility work needed for current releases.
Delete the unreferenced 2019 roadmap, which describes removed APIs and work that has since been implemented. Active work is tracked through GitHub issues.

Refs #181.
Separate test automation from documentation builds and run it for every pull request update.
@FrederikSchnack
FrederikSchnack added this pull request to stack #192 September 29, 2026 15:21
@FrederikSchnack FrederikSchnack mentioned this pull request Sep 29, 2026

@yguclu yguclu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good job! I have posted a few comments here and there.

BTW, I feel that this is the right PR to add an AUTHORS file, and probably a CHANGELOG too. What do you think?

Comment thread README.rst Outdated
Comment thread README.rst Outdated
Comment thread README.rst Outdated
Comment thread README.rst Outdated
Comment thread pyproject.toml Outdated
Comment thread old/form.py Outdated
Comment thread TODO.rst Outdated
Comment thread .github/workflows/testing.yml Outdated
Comment thread .github/workflows/testing.yml Outdated
FrederikSchnack and others added 6 commits September 29, 2026 20:38
Co-authored-by: Yaman Güçlü <yaman.guclu@gmail.com>
Co-authored-by: Yaman Güçlü <yaman.guclu@gmail.com>
Co-authored-by: Yaman Güçlü <yaman.guclu@gmail.com>
Co-authored-by: Yaman Güçlü <yaman.guclu@gmail.com>
@FrederikSchnack

Copy link
Copy Markdown
Contributor Author

BTW, I feel that this is the right PR to add an AUTHORS file, and probably a CHANGELOG too. What do you think?

I think that's a good idea! Do you want to add them?

@yguclu

yguclu commented Sep 29, 2026

Copy link
Copy Markdown
Member

BTW, I feel that this is the right PR to add an AUTHORS file, and probably a CHANGELOG too. What do you think?

I think that's a good idea! Do you want to add them?

Unfortunately, I do not think I will have the time before the developers' meeting tomorrow. If you want to draft them based on what we have in PSYDAC, I can probably review them

@FrederikSchnack

Copy link
Copy Markdown
Contributor Author

Unfortunately, I do not think I will have the time before the developers' meeting tomorrow. If you want to draft them based on what we have in PSYDAC, I can probably review them

Done in 7fde16b and f85db9a. For now, I only wrote Version 0.19.0 to 0.20.0. For consistency I also changed the README to md in b370bf8.

Comment thread AUTHORS Outdated
Comment thread README.md Outdated

@campospinto campospinto left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I had a quick look but it seems good to me, thanks!

FrederikSchnack and others added 2 commits October 1, 2026 09:57
Co-authored-by: Yaman Güçlü <yaman.guclu@gmail.com>
@yguclu yguclu changed the title Modernize the repository configuration Modernize repository configuration Oct 1, 2026
@yguclu
yguclu merged commit beab8bc into master Oct 1, 2026
13 checks passed
@yguclu
yguclu deleted the repo_maintenance branch October 1, 2026 08:29
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.

4 participants