Major Revision: CI Workflow Split into 2 Parts#631
Merged
DataBoySu merged 49 commits intoanmol098:masterfrom Jan 14, 2026
Merged
Major Revision: CI Workflow Split into 2 Parts#631DataBoySu merged 49 commits intoanmol098:masterfrom
DataBoySu merged 49 commits intoanmol098:masterfrom
Conversation
…to prevent CI stalling
* Update Dockerfile * Update ci_pr.yml
github-actions
bot
added
python
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Collaborator
Author
|
Pretty much done with edits, PR is ready. |
…aka-readme-stats into chore/pygithub-2.8.1
Max_CAP, Bug Fixes and Library Upgrade
github-actions
bot
added
documentation
This was
linked to
issues
Jan 14, 2026
Closed
|
README stats current output: 🐱 My GitHub Data
I'm a Night 🦉 📅 I'm Most Productive on Sunday 📊 This Week I Spent My Time On I Mostly Code in JavaScript Timeline You can use this website to view the generated base64 image. Last Updated on 14/01/2026 09:10:39 UTC |
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 16 out of 17 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
Author
|
Important information: New CI_PR will not be making comments anymore since the auto-provided Github token |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changelog:
Significant improvements to the CI/CD workflow and enhances testability and maintainability of the codebase.
Changes include splitting the CI workflow for push and pull request events, adding detection for documentation-only changes to skip unnecessary CI runs, improving environment variable handling for better error reporting and mocking, and ensuring assets directories are always created as needed.
There are also updates to support mock data for WakaTime API responses, which simplifies testing and PR previews.
CI/CD Workflow Improvements:
.github/workflows/ci.ymlfor push events (renamed toCI_MASTER) and.github/workflows/ci_pr.ymlfor pull requests (CI_PR). Each workflow now has tailored logic for detecting code changes versus documentation-only changes and will skip comprehensive CI if only docs are changed. This also includes improved permissions and artifact handling..github/workflows/codestyle.ymlnow also runs on pull requests, not just pushes.Mocking and Testability Enhancements:
MOCK_WAKATIMEandMOCK_DATA_DIRenvironment variables inEnvironmentManager, and updatingDownloadManagerto load mock data when enabled. This allows PR runs to use static data for previewing stats.DownloadManagerfor clarity and correctness, and optimized resource caching logic to prefer mock data when present.Environment and Error Handling:
EnvironmentManagerto usegetenvinstead ofenviron, provide better error messages for missing required secrets, and set default values where appropriate.File and Asset Management:
FileManagermethods to always create the assets directory when saving files or caching binaries, preventing errors when the directory does not exist.Minor Improvements and Cleanups:
locales/*in the build context.main.pyfor profile views and error messages.