-
Notifications
You must be signed in to change notification settings - Fork 41
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
PC-34428 BSR pro shard strategy #16221
Conversation
c25b9b6
to
91eadaa
Compare
Visit the preview URL for this PR (updated for commit b344e38): https://pc-pro-testing--pr16221-pc-34428-bsr-pro-sha-efuqbh54.web.app (expires Sun, 09 Feb 2025 09:27:07 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1 |
91eadaa
to
bfd6f05
Compare
@@ -218,7 +218,7 @@ jobs: | |||
GCP_EHP_WORKLOAD_IDENTITY_PROVIDER: ${{ secrets.GCP_EHP_WORKLOAD_IDENTITY_PROVIDER }} | |||
|
|||
test-pro: | |||
name: "Tests pro" | |||
name: "Units Tests" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
name: "Units Tests" | |
name: "Pro Units Tests" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai suggéré des noms plus explicites pour faciliter la lecture dans la page des metrics de performances.
Sinon cela me parait top
@@ -215,7 +215,7 @@ jobs: | |||
GCP_EHP_WORKLOAD_IDENTITY_PROVIDER: ${{ secrets.GCP_EHP_WORKLOAD_IDENTITY_PROVIDER }} | |||
|
|||
test-pro: | |||
name: "Tests pro" | |||
name: "Units Tests" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
name: "Units Tests" | |
name: "Pro Units Tests" |
@@ -47,32 +42,26 @@ jobs: | |||
- run: yarn lint:js | |||
- run: yarn lint:scss | |||
|
|||
tests-pro-unit-tests: | |||
run-tests: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pro-unit-tests
@@ -47,32 +42,26 @@ jobs: | |||
- run: yarn lint:js | |||
- run: yarn lint:scss | |||
|
|||
tests-pro-unit-tests: | |||
run-tests: | |||
name: "Unit tests" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
name: "Unit tests" | |
name: "Pro Unit tests" |
5caf59f
to
b344e38
Compare
b344e38
to
61f4738
Compare
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-34428
Vérifications