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

feat: Update argocd applicationset #101

Merged
merged 7 commits into from
Jan 5, 2025

Conversation

ghorbani-ali
Copy link
Contributor

This PR will fix:

  • Fix some issues about indent and manifest syntax
  • add templating options

@tumido tumido changed the title Update argocd applicationset feat: Update argocd applicationset Dec 10, 2024
Copy link
Owner

@tumido tumido left a comment

Choose a reason for hiding this comment

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

Hi @ghorbani-ali ! Thank you for your contribution! 🙌 It's so nice to see people contributing.

Can you please take a look at the comments I've made inline?

snippets/argoproj.json Outdated Show resolved Hide resolved
snippets/argoproj.json Outdated Show resolved Hide resolved
snippets/argoproj.json Outdated Show resolved Hide resolved
ghorbani-ali and others added 3 commits December 14, 2024 13:14
Co-authored-by: Tom Coufal <7453394+tumido@users.noreply.github.com>
Co-authored-by: Tom Coufal <7453394+tumido@users.noreply.github.com>
Co-authored-by: Tom Coufal <7453394+tumido@users.noreply.github.com>
@ghorbani-ali
Copy link
Contributor Author

ghorbani-ali commented Dec 14, 2024

Hi @tumido, it is my pleasure.
Thank you for your suggestion.
I updated my PR.

snippets/argoproj.json Outdated Show resolved Hide resolved
@ghorbani-ali ghorbani-ali requested a review from tumido December 18, 2024 14:26
@tumido
Copy link
Owner

tumido commented Jan 5, 2025

Hey, I'm sorry for a late response. I've spend a lot of time offline over the Christmas. Anyways, all looking good. Approving, merging and releasing a new version 🙂

Copy link
Owner

@tumido tumido left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@tumido tumido merged commit daec89f into tumido:master Jan 5, 2025
Copy link

github-actions bot commented Jan 5, 2025

🎉 This PR is included in version 1.2.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions bot added the released Pull requests that are included in a release label Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Pull requests that are included in a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants