-
Notifications
You must be signed in to change notification settings - Fork 6
Support Rack 3 (attempt #2) #96
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
Changes from 23 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
34dae27
Support Rack 3
ngan de654f3
Ignore dev env stuff
joemsak c7bb29d
Update Ruby, Node, Yarn, deps, replace apparition with cuprite
joemsak 948cfad
Remove docker container orphans on rake tasks
joemsak 800682a
Upgrade to latest envoy image
joemsak 2ca1de0
Update GH actions to use Ruby 3.4
joemsak f02ebb8
Remove frozen bundle line, not working in ruby 3.4
joemsak bdea892
Remove --remove-orphans flag
joemsak 21aca61
Remove 'chromium'; not needed with chrome service
joemsak ce7bee7
Update rubocop syntax
joemsak cc32774
Lint
joemsak 15d6d3b
WIP trying to upgrade protobuf, grpc, protoc, et al
joemsak 050b195
Use customized protoc image to upgrade protoc and the generators
joemsak 168df3b
Update envoy config
joemsak 8dbc3d8
Put the apparition code back
joemsak 57d2538
Rename workflow file, remove rspec
joemsak 33a5b8d
Try manual rake cmd
joemsak f4dd659
Okay, rspec is needed in CI
joemsak 6a4c9cf
Separate on lines
joemsak fa06b2b
Clean up rakefile
joemsak a7e1cf4
Replace apparition with selenium-webdriver, copied from gusto-sidekiq
joemsak 1e0356e
Fix env value type in docker-compose
joemsak a5b0eaf
AI-assisted spec fixes
mattbooks 3d5226b
Fix docker compose version in CI
mattbooks c0cac9c
Ignore cjs/mjs and undo test configs
mattbooks d6dd73d
Remove console log printing for feature tests
mattbooks 1744e59
Remove code for running specs without docker since that does not work
mattbooks b82f7b4
Clean up rakefile
mattbooks df4b80b
Fix rake constraint and update dependencies for CI
mattbooks 69349e2
Update .rubocop.yml
mattbooks 3e9bea2
Fix yarn workspace isolation for node-client (#97)
joemsak File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,6 +8,8 @@ coverage | |
|
|
||
| node_modules | ||
| .bundle | ||
| .yarn* | ||
|
|
||
| # Workspace | ||
| .idea | ||
| mise.toml | ||
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 2.7.8 | ||
| 3.4.2 |
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| FROM namely/protoc-all:latest | ||
|
|
||
| ENV DEBIAN_FRONTEND=noninteractive | ||
|
|
||
| RUN apt-get update && apt-get -y -o Dpkg::Options::="--force-confold" install \ | ||
| unzip \ | ||
| ruby | ||
|
|
||
| RUN gem install grpc-tools | ||
|
|
||
| RUN curl -fsSL https://deb.nodesource.com/setup_22.x -o nodesource_setup.sh \ | ||
| && bash nodesource_setup.sh \ | ||
| && apt-get -y -o Dpkg::Options::="--force-confold" install nodejs \ | ||
| && npm i -g npm \ | ||
| && npm i -g ts-protoc-gen --force \ | ||
| && npm i -g protoc-gen-js | ||
|
|
||
| RUN PROTOC_ZIP=protoc-28.0-linux-x86_64.zip \ | ||
| && curl -OL https://github.com/protocolbuffers/protobuf/releases/download/v28.0/$PROTOC_ZIP \ | ||
| && unzip -o $PROTOC_ZIP -d /usr/local bin/protoc \ | ||
| && unzip -o $PROTOC_ZIP -d /usr/local 'include/*' \ | ||
| && rm -f $PROTOC_ZIP | ||
|
|
||
| # Install modern protoc-gen-grpc-web plugin (compatible with grpc-web 1.5.0+) | ||
| RUN GRPC_WEB_VERSION=1.5.0 \ | ||
| && curl -OL https://github.com/grpc/grpc-web/releases/download/${GRPC_WEB_VERSION}/protoc-gen-grpc-web-${GRPC_WEB_VERSION}-linux-x86_64 \ | ||
| && mv protoc-gen-grpc-web-${GRPC_WEB_VERSION}-linux-x86_64 /usr/local/bin/protoc-gen-grpc-web \ | ||
| && chmod +x /usr/local/bin/protoc-gen-grpc-web | ||
|
|
||
| ENTRYPOINT [] |
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.