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

Sort results #75

Merged
merged 4 commits into from
Aug 28, 2024
Merged

Sort results #75

merged 4 commits into from
Aug 28, 2024

Conversation

lcssanches
Copy link
Contributor

Dealing with larger plans is harder if results are not sorted.

  • Replaced deprecated Linter
  • Added sorting logic
  • Added test to cover GetAllOutputChanges and GetAllResourceChanges

* Update terraform_state_test.go

* Update terraform_state_test.go

* Add tests to GetAllOutputChanges and GetAllResourceChanges

* Update terraform_state_test.go
@lcssanches
Copy link
Contributor Author

Hey @dineshba any idea when this can be released?

@dineshba dineshba merged commit be5c60e into dineshba:main Aug 28, 2024
1 check passed
@dineshba
Copy link
Owner

Sorry for the delay. I was in vacation. Just merged and release new version v0.3.11

@dineshba
Copy link
Owner

dineshba commented Sep 2, 2024

I am facing some issue with release. I will release in coming week. Sorry for the delay @lcssanches

@dineshba
Copy link
Owner

dineshba commented Oct 3, 2024

I was able to release yesterday. Sorry for the delay and once again thanks for the PR

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