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

docs(scripts): Update paths and commands in scripts README #1175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

skrashevich
Copy link
Contributor

This commit updates the README.md file in the scripts directory.
It's modifies the command to run goweight from a local script execution (./goweight) to using the latest version available on GitHub (go run github.com/paralin/goweight@latest). These modifications seem aimed at correcting paths and ensuring that the most updated tool version is used for better dependency management and weight analysis of the project.

@AlexxIT
Copy link
Owner

AlexxIT commented Jun 16, 2024

paralin? Why not original author?

@AlexxIT AlexxIT added the doubt label Jun 16, 2024
@skrashevich
Copy link
Contributor Author

paralin? Why not original author?

Because the original project hasn’t been updated for 5 years

@AlexxIT
Copy link
Owner

AlexxIT commented Jun 16, 2024

The 4 up-to-date commits in the above repository don't bring anything important

@skrashevich
Copy link
Contributor Author

original branch didn't pass environment variables to builder process. so, it's impossible to make calculation for another os/arch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants