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

chore: rename utils to podspec #78

Merged
merged 3 commits into from
Jun 28, 2024
Merged

chore: rename utils to podspec #78

merged 3 commits into from
Jun 28, 2024

Conversation

vsoch
Copy link
Member

@vsoch vsoch commented Jun 27, 2024

Problem: the "utils" package in fluence does not describe what it does, and these functions are hard for the developer to find.
Solution: rename to podspec to reflect this is what the package is oriented to do.

This will close #77

Problem: the "utils" package in fluence does not describe
what it does, and these functions are hard for the developer
to find.
Solution: rename to podspec to reflect this is what the
package is oriented to do.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Problem: LD_LIBRARY_PATH is not defined and is issuing
a warning during build.
Solution: remove the variable - I have confirmed it is
not defined in the base image.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Problem: the container builds are using an old Go, 1.19
Solution: update to 1.21 to match the rest of CI recipes.
Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@vsoch
Copy link
Member Author

vsoch commented Jun 27, 2024

Amazing - I just moved a change between commits and I didn't break anything. There must be a lot of pigs flying outside today.

@vsoch vsoch requested a review from milroy June 27, 2024 22:22
@vsoch
Copy link
Member Author

vsoch commented Jun 27, 2024

No rush to review these @milroy - please take care of the 🔥 first.

Copy link
Collaborator

@cmisale cmisale left a comment

Choose a reason for hiding this comment

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

lgtm, that's definitely a much better name

@vsoch
Copy link
Member Author

vsoch commented Jun 28, 2024

Thanks @cmisale ! Is it OK if I ping you for future reviews? I wasn't sure if you had the bandwidth since the project was coming to a close.

@vsoch vsoch merged commit 1ade802 into main Jun 28, 2024
8 checks passed
@vsoch vsoch deleted the chore/reorganize-utils branch June 28, 2024 01:16
@cmisale
Copy link
Collaborator

cmisale commented Jun 28, 2024

Absolutely! I'm incredibly quiet but I follow. I always take a look at every PR added me as reviewer even tho some are very big and cannot look at everything with the care I'd like..

@vsoch
Copy link
Member Author

vsoch commented Jun 28, 2024

Thank you 🙏 I didn't know! I apologize for not asking for your review before - I didn't want to burden you.

@vsoch
Copy link
Member Author

vsoch commented Jun 28, 2024

I'll try to keep them as small / scoped as possible - that first one was... 🤯

@cmisale
Copy link
Collaborator

cmisale commented Jun 28, 2024

No need to apologize :| I've been absent for some time, so it's totally understandable. And haha yes, the first one was huge lol

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.

Rename src/fluence/utils to something I'll remember
2 participants