Skip to content

Conversation

@wreiske
Copy link
Contributor

@wreiske wreiske commented May 20, 2025

✅ (ci): add GitHub Actions workflow for running tests on push and PR
🔧 (gitignore): add .env to ignore environment config files
♻️ (blaze/attrs): optimize attribute updates to avoid redundant DOM changes
⬆️ (test-app): upgrade Meteor release and package versions for compatibility
⬆️ (test-app): update npm dependencies to latest versions

wreiske added 6 commits May 19, 2025 20:27
🔧 (gitignore): add .env to ignore environment config files
♻️ (blaze/attrs): optimize attribute updates to avoid redundant DOM changes
⬆️ (test-app): upgrade Meteor release and package versions for compatibility
⬆️ (test-app): update npm dependencies to latest versions
✅ (blaze-tests.yml): improve Meteor install and test process in CI
🔧 (blaze-tests.yml): update Meteor install to use latest and set PATH via GITHUB_PATH
🔧 (blaze-tests.yml): remove ci.sh and handle all test steps in workflow file
…eadiness check

🐛 (blaze-tests.yml): fix readiness check to use log file instead of file descriptor
✅ (blaze-tests.yml): add failure handling and log output if test-in-console is not ready
…event

💡 (puppeteerRunner.js): improve browser console logging for better debugging
@jankapunkt
Copy link
Collaborator

@wreiske thanks a lot for the love for Blaze! Let me know when its ready to review or if you need support.

@radekmie
Copy link
Collaborator

If possible, please split it into two (or more) PRs. It's weird to end up in a "CI improvements" PR when doing a bisect 😅

@wreiske
Copy link
Contributor Author

wreiske commented May 20, 2025

If possible, please split it into two (or more) PRs. It's weird to end up in a "CI improvements" PR when doing a bisect 😅

Amen! I had an itch I needed to scratch last night and tend to do a lot of things on a single PR. I'll get these split out into bite sized pieces today! 🙏

@StorytellerCZ StorytellerCZ requested a review from jankapunkt May 20, 2025 14:50
…n attrs.js

🔧 (test-app): remove unused test:ci script from package.json
@wreiske
Copy link
Contributor Author

wreiske commented May 22, 2025

#477 for CI improvements
#478 for the experimental change

@jankapunkt
Copy link
Collaborator

I am closing this in favor of #477 and #478

@jankapunkt jankapunkt closed this May 22, 2025
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.

3 participants