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.

Sign up for GitHub

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

Jump to bottom

code-Update Dockerfile #7718

Merged
merged 2 commits into from
Dec 18, 2024
Merged

code-Update Dockerfile #7718

merged 2 commits into from
Dec 18, 2024

Conversation

romashka-btc
Copy link
Contributor

Fix: Sorted package names alphanumerically in Dockerfile

What was changed

  • Sorted package names alphanumerically in the RUN instruction within the Dockerfile for better readability and maintainability.

Why these changes were made

  • Sorting package names improves clarity, makes dependency management easier, and ensures consistency within the Dockerfile.

Additional Notes

  • This change focuses solely on improving code organization and does not alter functionality.

@romashka-btc
code-Update Dockerfile
0915dfc 
This adheres to Dockerfile best practices for maintainability and consistency.
damiannolan
damiannolan approved these changes Dec 18, 2024
View reviewed changes
Copy link
Contributor

@damiannolan damiannolan left a comment

Choose a reason for hiding this comment

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

Goodluck with airdrops sir 🫡

@damiannolan damiannolan merged commit d257acb into cosmos:main Dec 18, 2024
70 of 75 checks passed
@romashka-btc
Copy link
Contributor Author

Goodluck with airdrops sir 🫡

thank you bro !

gjermundgaraba pushed a commit that referenced this pull request Feb 5, 2025
@romashka-btc @damiannolan @gjermundgaraba
code-Update Dockerfile (#7718)
f988352 
This adheres to Dockerfile best practices for maintainability and consistency.

Co-authored-by: Damian Nolan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damiannolan damiannolan damiannolan approved these changes

@AdityaSripal AdityaSripal Awaiting requested review from AdityaSripal AdityaSripal is a code owner

@DimitrisJim DimitrisJim Awaiting requested review from DimitrisJim

@gjermundgaraba gjermundgaraba Awaiting requested review from gjermundgaraba gjermundgaraba is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@romashka-btc @damiannolan