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

Ride vm and compiler. #438

Draft
wants to merge 61 commits into
base: master
Choose a base branch
from
Draft

Ride vm and compiler. #438

wants to merge 61 commits into from

Conversation

Frozen
Copy link
Contributor

@Frozen Frozen commented Mar 18, 2021

No description provided.

Makefile Outdated
@@ -11,7 +11,7 @@ export GO111MODULE=on

.PHONY: vendor vetcheck fmtcheck clean build gotest

all: vendor vetcheck fmtcheck build gotest mod-clean
Copy link
Contributor

Choose a reason for hiding this comment

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

It's better to skip a package instead of removing fmtcheck at all in case if it's hard to fix style and stuff

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've tried all possible ways - skip, ignore, fix, upgrade. Nothing helps.

@alexeykiselev alexeykiselev added do not merge The PR is not ready to be merged wip This is a WIP, should not be merged right away labels Apr 26, 2021
@nickeskov nickeskov marked this pull request as draft May 21, 2022 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge The PR is not ready to be merged wip This is a WIP, should not be merged right away
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants