Skip to content

chore: Remove /infra folder#24462

Merged
volnei merged 1 commit into
mainfrom
chore/remove-infra-folder
Oct 14, 2025
Merged

chore: Remove /infra folder#24462
volnei merged 1 commit into
mainfrom
chore/remove-infra-folder

Conversation

@keithwillcode
Copy link
Copy Markdown
Contributor

@keithwillcode keithwillcode commented Oct 14, 2025

This is outdated and not used.

@keithwillcode keithwillcode added this to the v5.8 milestone Oct 14, 2025
@keithwillcode keithwillcode self-assigned this Oct 14, 2025
@keithwillcode keithwillcode requested a review from a team as a code owner October 14, 2025 18:53
@keithwillcode keithwillcode added the ci area: CI, DX, pipeline, github actions label Oct 14, 2025
@keithwillcode keithwillcode added core area: core, team members only foundation labels Oct 14, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 14, 2025

Walkthrough

This change removes infra-related components and references. The infra/**/* entry is deleted from .github/CODEOWNERS. The infra/README.md file is removed. The infra/docker/api/Dockerfile is deleted, eliminating the prior multi-stage Docker build for the API. In package.json, three scripts are removed: docker-build-api, docker-run-api, and docker-stop-api. No other files are modified, and no application logic is changed.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly and concisely summarizes the primary change by stating the removal of the infra folder and uses a standard chore prefix, making it straightforward for collaborators to understand the main purpose of the pull request.
Description Check ✅ Passed The description succinctly indicates that the infra folder is outdated and unused, which aligns with the removal of that folder in the changeset and is therefore relevant to the pull request’s content.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-infra-folder

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between e26a57b and cb0eb9e.

📒 Files selected for processing (4)
  • .github/CODEOWNERS (0 hunks)
  • infra/README.md (0 hunks)
  • infra/docker/api/Dockerfile (0 hunks)
  • package.json (0 hunks)
💤 Files with no reviewable changes (4)
  • infra/README.md
  • package.json
  • .github/CODEOWNERS
  • infra/docker/api/Dockerfile

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@volnei volnei enabled auto-merge (squash) October 14, 2025 19:00
@volnei volnei merged commit 3b19c06 into main Oct 14, 2025
77 of 82 checks passed
@volnei volnei deleted the chore/remove-infra-folder branch October 14, 2025 19:23
@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

akarsh-jain-790 pushed a commit to akarsh-jain-790/cal.com that referenced this pull request Mar 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci area: CI, DX, pipeline, github actions core area: core, team members only foundation ready-for-e2e size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants