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

refactor: import plugin default configs with go:embed #614

Merged
merged 4 commits into from
May 27, 2022

Conversation

iyear
Copy link
Member

@iyear iyear commented May 26, 2022

refactor: import plugin default configs with go:embed

Pre-Checklist

Note: please complete ALL items in the following checklist.

  • I have read through the CONTRIBUTING.md documentation.
  • My code has the necessary comments and documentation (if needed).
  • I have added relevant tests

Description

Refactor the default configs reading process and use embed to import all default configs.

And #597 I haven't done it yet, the requirements are still not clear to me.

Related Issues

#603

New Behavior (screenshots if needed)

@daniel-hutao daniel-hutao self-requested a review May 26, 2022 09:40
@iyear iyear marked this pull request as ready for review May 26, 2022 10:08
@iyear iyear mentioned this pull request May 26, 2022
3 tasks
internal/pkg/show/config/default.yaml Outdated Show resolved Hide resolved
internal/pkg/show/config/plugins/argocdapp.yaml Outdated Show resolved Hide resolved
@daniel-hutao daniel-hutao merged commit c6b4aba into devstream-io:main May 27, 2022
@daniel-hutao
Copy link
Member

Good Job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants