Skip to content

chore: add Replexica env to ci tests#17970

Merged
keithwillcode merged 3 commits into
mainfrom
add-replexica-api-key-ci
Dec 4, 2024
Merged

chore: add Replexica env to ci tests#17970
keithwillcode merged 3 commits into
mainfrom
add-replexica-api-key-ci

Conversation

@ThyMinimalDev
Copy link
Copy Markdown
Contributor

@ThyMinimalDev ThyMinimalDev commented Dec 3, 2024 •
edited
Loading

What does this PR do?

CI tests failing because of missing env var for REPLEXICA
example failure here : https://github.com/calcom/cal.com/actions/runs/12140460762/job/33851121538?pr=17959#step:8:240

changes to translation with REPLEXICA are now bundled inside platform libraries which is triggering the issue

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

@dosubot dosubot Bot added ci area: CI, DX, pipeline, github actions ❗️ .env changes contains changes to env variables labels Dec 3, 2024
@keithwillcode keithwillcode added core area: core, team members only platform Anything related to our platform plan labels Dec 3, 2024
@ThyMinimalDev ThyMinimalDev added ready-for-e2e and removed ❗️ .env changes contains changes to env variables ci area: CI, DX, pipeline, github actions labels Dec 3, 2024
@graphite-app graphite-app Bot requested a review from a team December 3, 2024 14:14
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Dec 3, 2024

Graphite Automations

"Add platform team as reviewer" took an action on this PR • (12/03/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 3, 2024 •
edited
Loading

E2E results are ready!

keithwillcode
keithwillcode requested changes Dec 3, 2024
View reviewed changes
Comment thread .github/workflows/e2e-api-v2.yml Outdated
@github-actions github-actions Bot marked this pull request as draft December 3, 2024 18:16
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 3, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Dec 4, 2024 9:09am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Dec 4, 2024 9:09am

@ThyMinimalDev ThyMinimalDev marked this pull request as ready for review December 3, 2024 19:00
@dosubot dosubot Bot added ci area: CI, DX, pipeline, github actions ❗️ .env changes contains changes to env variables labels Dec 3, 2024
@keithwillcode keithwillcode changed the title chore: add REPLEXICA env to ci tests chore: add Replexica env to ci tests Dec 4, 2024
@keithwillcode keithwillcode enabled auto-merge (squash) December 4, 2024 09:22
@keithwillcode keithwillcode merged commit b92db55 into main Dec 4, 2024
@keithwillcode keithwillcode deleted the add-replexica-api-key-ci branch December 4, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@keithwillcode keithwillcode keithwillcode approved these changes

@zomars zomars Awaiting requested review from zomars

Assignees

No one assigned

Labels

ci area: CI, DX, pipeline, github actions core area: core, team members only ❗️ .env changes contains changes to env variables platform Anything related to our platform plan ready-for-e2e

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ThyMinimalDev @keithwillcode