-
Notifications
You must be signed in to change notification settings - Fork 0
Dev #43
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.
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
Merged
Merged
Dev #43
Conversation
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
…cache compatibility - Move 'Add MySQL to PATH' step before 'Install MySQL via SpinDB' step - Move 'Add SQLite to PATH' steps before 'Install SQLite via SpinDB' steps in both SQLite and CLI E2E jobs - Update comments to clarify PATH setup is needed BEFORE install check for cache hits - Add echo statements to confirm when paths are added - Ensures cached binaries are found in PATH before SpinDB checks for existing installations
…mpatibility - Add 'Link MySQL (macOS)' step before 'Install MySQL via SpinDB' step - Check if MySQL is installed in /opt/homebrew/Cellar/mysql before linking - Use brew link with error suppression to handle already-linked cases - Add echo statement to confirm linking status - Ensures cached MySQL binaries are properly linked before SpinDB checks for existing installations
- Remove 'Cache Homebrew packages (macOS)' step from MySQL integration tests - Remove 'Link MySQL (macOS)' step that handled cached installations - Add NOTE comment explaining Homebrew dependencies (protobuf, zlib) are dynamically linked and not included in Cellar cache, causing dyld errors - Fresh install is more reliable than partial caching on macOS
- Change TODO comment from "testing CI on dev branch" to "testing CI" for broader applicability - Remove branch-specific reference to make comment more generic
…lel execution - Remove `needs: unit-tests` from test-postgresql, test-mysql, test-sqlite, and test-cli-e2e jobs - Update unit tests job comment from "Run first for fast feedback" to generic description - Allows all test jobs to run in parallel instead of sequentially waiting for unit tests - Reduces overall CI workflow execution time
…st optimization - Remove push trigger for feature/test-speed branch from CI workflow - Keep commented dev branch trigger available for future testing - Maintain active pull_request and workflow_dispatch triggers
Feature/test speed
- Bump version from 0.11.1 to 0.11.2 in package.json - Add 0.11.2 release entry to CHANGELOG.md with CI parallelization, Node.js 22 update, and macOS test concurrency fix - Update package description to include SQLite support
…anagement - Rewrite description to highlight "Run local" instead of "Spin up" - Emphasize "without Docker" and position as "lightweight DBngin/Postgres.app alternative" - Add "interactive CLI" to description for better feature visibility - Remove duplicate documentation sections from CHANGELOG.md (0.11.1 and 0.7.1)
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.
No description provided.