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

Fix Version Detection: whitespace not trimmed #300

Closed
codezninja opened this issue Sep 16, 2020 · 3 comments · Fixed by #301
Closed

Fix Version Detection: whitespace not trimmed #300

codezninja opened this issue Sep 16, 2020 · 3 comments · Fixed by #301
Labels
bug Something isn't working
Milestone

Comments

@codezninja
Copy link
Contributor

It looks like when consolidating we removed version trimming. Before the readFile function read the file and removed whitespace. Should we add that logic back OR maybe document it in the release?

version = version ?: Jenkinsfile.original.readFile(TERRAFORM_VERSION_FILE) ?: DEFAULT_VERSION

@kmanning
Copy link
Collaborator

Whomp. I think that broke during a refactor. The readFile method was getting copy pasted everywhere, and I'm trying to consolidate it into Jenkinsfile. https://github.com/manheim/terraform-pipeline/blob/master/src/Jenkinsfile.groovy#L120-L127

@codezninja
Copy link
Contributor Author

hmm based on that it should be trimming it. Let me see if I can find the issue.

@kmanning kmanning added the bug Something isn't working label Sep 16, 2020
@kmanning
Copy link
Collaborator

Here's the PR that refactored the code and broke the Plugin - https://github.com/manheim/terraform-pipeline/pull/298/files#diff-2724a09cdd048285c15140f07b77fb5dR33

@kmanning kmanning changed the title Version Detection Fix Version Detection: whitespace not trimmed Sep 16, 2020
@codezninja codezninja added this to the v5.12 milestone Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants