Skip to content

Reapply updated Copier templates and streamline developer workflow#97

Merged
AndrewSazonov merged 8 commits intodevelopfrom
reapply-templates
Feb 6, 2026
Merged

Reapply updated Copier templates and streamline developer workflow#97
AndrewSazonov merged 8 commits intodevelopfrom
reapply-templates

Conversation

@AndrewSazonov
Copy link
Member

@AndrewSazonov AndrewSazonov commented Feb 5, 2026

This PR reapplies the latest EasyScience Copier templates to align the repository with updated project structure, CI workflows, and configuration.

As part of this maintenance work, pre-commit and pre-push hooks are temporarily disabled to simplify local development. Instead, developers are expected to:

  • run pixi check to manually run code quality checks and unit tests (at least before opening a PR),
  • run pixi fix to apply all automatic formatting and lint fixes.

This keeps the workflow lightweight during active development while preserving consistent quality checks.

To ensure these checks are not forgotten, the Code quality checks workflow is still executed on every push and pull request, so any remaining issues must be addressed before merging.

@AndrewSazonov AndrewSazonov added the [scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH) label Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@88af3d9). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop      #97   +/-   ##
==========================================
  Coverage           ?   97.91%           
==========================================
  Files              ?       32           
  Lines              ?     1825           
  Branches           ?      296           
==========================================
  Hits               ?     1787           
  Misses             ?       21           
  Partials           ?       17           
Flag Coverage Δ
integration 0.00% <ø> (?)
unittests 97.91% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@henrikjacobsenfys henrikjacobsenfys left a comment

Choose a reason for hiding this comment

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

LGTM

I may want to get rid of some icons in the readme and docs since they're overused by AI and may therefore send the wrong signal to users. But that's for general cleanup which will be later

@AndrewSazonov
Copy link
Member Author

I understand the concern. Just to clarify, these are standard Unicode icons that are supported everywhere. The main idea is to make the lists easier to read and quicker to recognize visually.

For now, I wouldn’t suggest changing the README. The current version is auto-generated as a minimal and clean README, with the most important links and a consistent visual style across all EasyScience projects. This is based on the project configuration in the home repository (in this case easyscience/dynamics).

Later, when this becomes an ADR suggestion, I’d be happy to collect feedback and discuss things like icon usage there, so we can make a common decision for the whole organization instead of changing things project by project 🙂

@henrikjacobsenfys
Copy link
Member

I understand the concern. Just to clarify, these are standard Unicode icons that are supported everywhere. The main idea is to make the lists easier to read and quicker to recognize visually.

For now, I wouldn’t suggest changing the README. The current version is auto-generated as a minimal and clean README, with the most important links and a consistent visual style across all EasyScience projects. This is based on the project configuration in the home repository (in this case easyscience/dynamics).

Later, when this becomes an ADR suggestion, I’d be happy to collect feedback and discuss things like icon usage there, so we can make a common decision for the whole organization instead of changing things project by project 🙂

Yeah, let's discuss such details it when the ADR suggestion is there. It's so much better than what I had, and the consistency between projects is really nice.

@AndrewSazonov AndrewSazonov merged commit 33b093b into develop Feb 6, 2026
36 checks passed
henrikjacobsenfys pushed a commit that referenced this pull request Feb 6, 2026
* Update pixi.lock

* Reapply templates from v0.4.0

* pixi run fix-all

* Switch to manual pre-commit and simplify tasks

* Enable PyPI Trusted Publishing via OIDC

* Reorder imports and widget magic in notebooks

* Normalize notebook cell IDs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants