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

Bump porter to 1.0.1 #183

Merged
merged 3 commits into from
Oct 20, 2022
Merged

Bump porter to 1.0.1 #183

merged 3 commits into from
Oct 20, 2022

Conversation

carolynvs
Copy link
Member

@carolynvs carolynvs commented Oct 17, 2022

  • Update to Go 1.19
  • Build against porter 1.0.1
  • Update the schema of resources for the latest version of porter

@carolynvs carolynvs force-pushed the bump-porter branch 2 times, most recently from c7cff6a to 314473e Compare October 17, 2022 16:47
* Update to Go 1.19
* Build against porter 1.0.1

Signed-off-by: Carolyn Van Slyck <[email protected]>
@carolynvs carolynvs marked this pull request as ready for review October 17, 2022 22:45
@@ -405,7 +405,7 @@ type PorterCfgOpts struct {
SecretsNamespace string
}

//NewPorterCofnig GetDefaultPorterConfig - DefaultStorage, DefaultSecretsPlugin and DefaultStorageConfig set. We tweak DefaultSecrets and SecretsConfig as needed
// NewPorterCofnig GetDefaultPorterConfig - DefaultStorage, DefaultSecretsPlugin and DefaultStorageConfig set. We tweak DefaultSecrets and SecretsConfig as needed
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we're in here touching this might as well fix the typo :)

Copy link
Member Author

Choose a reason for hiding this comment

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

Hah, oops I am just awful at spotting typos. 👍

Signed-off-by: Carolyn Van Slyck <[email protected]>
@carolynvs carolynvs merged commit f01c944 into getporter:main Oct 20, 2022
@carolynvs carolynvs deleted the bump-porter branch October 20, 2022 20:20
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