Skip to content
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

Feature/6 automatic config - workflows #9

Merged
merged 6 commits into from
Nov 17, 2024
Merged

Conversation

adrrf
Copy link
Contributor

@adrrf adrrf commented Nov 16, 2024

This pull request includes updates to the .github/CODEOWNERS file and adds a new GitHub Actions workflow for syncing OpenAPI specs. Additionally, a new script sync-openapi.sh is introduced to fetch OpenAPI specifications for various services.

Key changes:

  • Updates to code owners:

    • .github/CODEOWNERS: Changed the code owner from @alvarobernal2412 to @AntonioRodriguezRuiz.
  • New GitHub Actions workflow:

    • .github/workflows/sync-openapi.yml: Added a new workflow to sync OpenAPI specs, which includes steps for checking out the repository, running the sync-openapi.sh script, and committing and pushing changes.
  • New script for syncing OpenAPI specs:

    • sync-openapi.sh: Created a script to fetch OpenAPI specifications for a list of services from a specified base URL and save them in the docs directory. The script also logs the date of the last update.

@github-actions github-actions bot added 🏠 Github Configuration Indicates the change is related to github settings 🚀 CI/CD Indicates the change is related to CI/CD workflows size/s labels Nov 16, 2024
@adrrf adrrf marked this pull request as ready for review November 17, 2024 09:59
@adrrf adrrf changed the title Feature/6 automatic config Feature/6 automatic config - workflows Nov 17, 2024
@adrrf adrrf self-assigned this Nov 17, 2024
Base automatically changed from feature/rust to main November 17, 2024 10:04
@github-actions github-actions bot removed the size/s label Nov 17, 2024
Copy link
Contributor

@alvarobernal2412 alvarobernal2412 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks nice but it would be perfect with some fixes!

.github/workflows/sync-openapi.yml Outdated Show resolved Hide resolved
.github/workflows/sync-openapi.yml Outdated Show resolved Hide resolved
.github/workflows/sync-openapi.yml Outdated Show resolved Hide resolved
@github-actions github-actions bot added 📦 Dependencies Pull requests that update a dependency file ⚙️ Configuration Indicates the scope is related to the configuration 😖 size/l Pull request size L labels Nov 17, 2024
Copy link
Contributor

@alvarobernal2412 alvarobernal2412 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!😈

@github-actions github-actions bot added 😎 size/m Pull request size M and removed 📦 Dependencies Pull requests that update a dependency file ⚙️ Configuration Indicates the scope is related to the configuration 😖 size/l Pull request size L labels Nov 17, 2024
Copy link
Contributor

@AntonioRodriguezRuiz AntonioRodriguezRuiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@adrrf adrrf merged commit 8568248 into main Nov 17, 2024
2 checks passed
@adrrf adrrf deleted the feature/6-automatic-config branch November 17, 2024 10:28
@adrrf adrrf restored the feature/6-automatic-config branch November 17, 2024 10:28
@AntonioRodriguezRuiz AntonioRodriguezRuiz deleted the feature/6-automatic-config branch November 17, 2024 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 CI/CD Indicates the change is related to CI/CD workflows 🏠 Github Configuration Indicates the change is related to github settings 😎 size/m Pull request size M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants