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: terraform subdirectory access in module source #209

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

mattp0
Copy link
Contributor

@mattp0 mattp0 commented Oct 14, 2024

This was introduced in previous v18.5.0.0 release. Plan to fix: delete v18.5.0.0 release and then re-generate the release after this is merged.

@mattp0
Copy link
Contributor Author

mattp0 commented Oct 14, 2024

Was reported that this was an issue by NSIDC in slack.

Copy link
Contributor

@reweeden reweeden left a comment

Choose a reason for hiding this comment

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

You could also do a v18.5.0.1. But fine with me to regenerate it I guess.

@mattp0
Copy link
Contributor Author

mattp0 commented Oct 14, 2024

You could also do a v18.5.0.1. But fine with me to regenerate it I guess.

My thought is that v18.5.0.0 is broken and this is a fix to it? but I can go either way. Just one I have to update a changelog

@mattp0 mattp0 merged commit ab9afad into master Oct 14, 2024
2 checks passed
@mattp0 mattp0 deleted the mrp/fix/terraform-subdirectory branch October 14, 2024 18:35
@reweeden
Copy link
Contributor

In a real package repository such as pypi you wouldn't have the ability to modify a version once you released it because someone could have built off of it already even with all it's issues. You would just upload a newer version with the bugfix and bump the bugfix version number, that's what it's there for.

For this, it seems like it's unusable in its current state? So not really an issue to completely delete and remake it.

@reweeden
Copy link
Contributor

I guess just keep in mind that this wouldn't be standard practice for software releases generally.

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.

3 participants