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

housekeeping: Upgrade tooling and deps #237

Merged
merged 2 commits into from
Jan 8, 2024
Merged

housekeeping: Upgrade tooling and deps #237

merged 2 commits into from
Jan 8, 2024

Conversation

juliaogris
Copy link
Member

@juliaogris juliaogris commented Jan 8, 2024

Upgrade tooling and go dependencies. I tried to write a make target for this,
but there seem to be too many manual steps involved - e.g. optionally
updating go.mod file when go minor version changes, or manually installing
most recent firebase and node version.

After the upgrade is done, finally pin the prettier version by providing
package.json file.

Upgrade tools with

	 hermit upgrade

Upgrade node manually

	hermit install node-21.5.0

Again I'm not certain why hermit didn't do this, something todo with no
explicit latest. I should get to the bottom of this.

Upgrade golangci-lint config .golangci.yaml to not fail linting.

Upgrade go.mod/go.sum to use latest versions of dependencies with

	go get -u
	go mod tidy
Pin prettier version by using a package.json file. Instead of polluting the
root directory of this repo further, we have installed prettier with

	npm install --prefix .hermit/node --save-dev prettier

Add .hermit/node/package.json and .hermit/node/package-lock.json for tool
version tracking.
Copy link
Member

@camh- camh- left a comment

Choose a reason for hiding this comment

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

🥦 I have to assume the npx stuff is right and works. I could do some research, but sorry, couldn't be bothered :-|

@juliaogris juliaogris merged commit c0dbaf0 into main Jan 8, 2024
3 checks passed
@juliaogris juliaogris deleted the e2e branch January 8, 2024 21:10
@juliaogris juliaogris added this to the 🚸 Learn v0.2.0 milestone Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants