Skip to content

Makefile: replace enumer upstream with dmarkham's#242

Merged
lbajolet-hashicorp merged 1 commit intomainfrom
change_enumer_fork_upstream
Jun 3, 2024
Merged

Makefile: replace enumer upstream with dmarkham's#242
lbajolet-hashicorp merged 1 commit intomainfrom
change_enumer_fork_upstream

Conversation

@lbajolet-hashicorp
Copy link
Copy Markdown
Contributor

Since the enumer implementation we used hadn't been updated for 5+ years, this didn't work with recent linux/go versions, and enumer crashed while attempting to parse/analyse the source files.

There's another alternative on Github, forked from the one we used, which seems more maintained now, and does produce the expected files in the SDK.

Since the enumer implementation we used hadn't been updated for 5+
years, this didn't work with recent linux/go versions, and enumer
crashed while attempting to parse/analyse the source files.

There's another alternative on Github, forked from the one we used,
which seems more maintained now, and does produce the expected files in
the SDK.
@lbajolet-hashicorp lbajolet-hashicorp added tech-debt Issues and pull requests related to addressing technical debt or improving the codebase dependencies Pull requests that update a dependency file labels Jun 3, 2024
@lbajolet-hashicorp lbajolet-hashicorp requested a review from a team as a code owner June 3, 2024 14:35
@lbajolet-hashicorp lbajolet-hashicorp merged commit b0fc5d9 into main Jun 3, 2024
@lbajolet-hashicorp lbajolet-hashicorp deleted the change_enumer_fork_upstream branch June 3, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file tech-debt Issues and pull requests related to addressing technical debt or improving the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants