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.

Sign up for GitHub

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

Jump to bottom

Feature/ci #182

Merged
merged 11 commits into from
Dec 27, 2024
Merged

Feature/ci #182

merged 11 commits into from
Dec 27, 2024

Conversation

zacdezgeo
Copy link
Collaborator

What I Changed

  • Add CI for unit tests
  • Rename sentinel file to example
  • Add pre-commit and integrate to CI
  • Implement fixes to ruff suggestions
  • Add ignore comments for QA from ruff
  • Update relative imports

How to test it

  • Run examples within local version of QGIS
  • See github action execution

Other notes

@zacdezgeo zacdezgeo requested review from gena and alukach December 20, 2024 22:12
@zacdezgeo zacdezgeo self-assigned this Dec 20, 2024
gena
gena approved these changes Dec 27, 2024
View reviewed changes
Copy link
Collaborator

@gena gena left a comment

Choose a reason for hiding this comment

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

Thanks for the cleanup

@gena gena merged commit c71b244 into master Dec 27, 2024
4 checks passed
@zacdezgeo zacdezgeo deleted the feature/ci branch January 2, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gena gena gena approved these changes

@alukach alukach Awaiting requested review from alukach

Assignees

@zacdezgeo zacdezgeo

Labels
enhancement
Projects
None yet
Milestone
No milestone
2 participants
@zacdezgeo @gena