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

refactor!: remove gno build command #1297

Merged
merged 5 commits into from
Nov 16, 2023

Conversation

tbruyelle
Copy link
Contributor

@tbruyelle tbruyelle commented Oct 26, 2023

Closes: #1242

In favor of gno precompile -gobuild.

As seen in the tests written for gno build in this PR #1103, the command's behavior is kinda unexpected. It actually doesn't care about the syntax of gno files and only relies on go files found in the path passed as an argument.

Because of that, I chose to name the flag -gobuild instead of -build because I found that represents better what's really happening when this flag is provided.

As mentioned in the linked issue, gno build will be re-implemented in the future with an other behavior.

Contributors' checklist...
  • Added new tests, or not needed, or not feasible
  • Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
  • Updated the official documentation or not needed
  • No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
  • Added references to related issues and PRs
  • Provided any useful hints for running manual tests
  • Added new benchmarks to generated graphs, if any. More info here.

@tbruyelle tbruyelle requested review from jaekwon, moul and a team as code owners October 26, 2023 02:17
@github-actions github-actions bot added the 📦 🤖 gnovm Issues or PRs gnovm related label Oct 26, 2023
@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (7d66398) 55.94% compared to head (ad1df0a) 55.98%.

Files Patch % Lines
gnovm/cmd/gno/precompile.go 81.25% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1297      +/-   ##
==========================================
+ Coverage   55.94%   55.98%   +0.03%     
==========================================
  Files         420      420              
  Lines       65415    65443      +28     
==========================================
+ Hits        36596    36637      +41     
+ Misses      25961    25942      -19     
- Partials     2858     2864       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tbruyelle tbruyelle changed the title BREAKING CHANGE: remove gno build command refactor!: remove gno build command Oct 26, 2023
@tbruyelle tbruyelle force-pushed the tbruyelle/refactor/remove-gno-build branch from 101d2bb to 41a602d Compare October 26, 2023 14:52
@tbruyelle tbruyelle force-pushed the tbruyelle/refactor/remove-gno-build branch from 41a602d to ad1df0a Compare November 16, 2023 17:12
@thehowl thehowl merged commit 0c68394 into gnolang:master Nov 16, 2023
186 checks passed
thehowl pushed a commit that referenced this pull request Jan 10, 2024
- Remove `gno build` as it doesn't exist anymore (see: #1297)
- s/go-binary/gofmt

<details><summary>Contributors' checklist...</summary>

- [x] Added new tests, or not needed, or not feasible
- [x] Provided an example (e.g. screenshot) to aid review or the PR is
self-explanatory
- [x] Updated the official documentation or not needed
- [x] No breaking changes were made, or a `BREAKING CHANGE: xxx` message
was included in the description
- [x] Added references to related issues and PRs
- [x] Provided any useful hints for running manual tests
- [x] Added new benchmarks to [generated
graphs](https://gnoland.github.io/benchmarks), if any. More info
[here](https://github.com/gnolang/gno/blob/master/.benchmarks/README.md).
</details>
gfanton pushed a commit to moul/gno that referenced this pull request Jan 18, 2024
- Remove `gno build` as it doesn't exist anymore (see: gnolang#1297)
- s/go-binary/gofmt

<details><summary>Contributors' checklist...</summary>

- [x] Added new tests, or not needed, or not feasible
- [x] Provided an example (e.g. screenshot) to aid review or the PR is
self-explanatory
- [x] Updated the official documentation or not needed
- [x] No breaking changes were made, or a `BREAKING CHANGE: xxx` message
was included in the description
- [x] Added references to related issues and PRs
- [x] Provided any useful hints for running manual tests
- [x] Added new benchmarks to [generated
graphs](https://gnoland.github.io/benchmarks), if any. More info
[here](https://github.com/gnolang/gno/blob/master/.benchmarks/README.md).
</details>
@tbruyelle tbruyelle deleted the tbruyelle/refactor/remove-gno-build branch February 22, 2024 12:20
harry-hov added a commit to harry-hov/gno that referenced this pull request Apr 8, 2024
As `gno build` was removed in gnolang#1297 in favor of `gno transpile -gobuild`.
harry-hov added a commit to harry-hov/gno that referenced this pull request Apr 8, 2024
As `gno build` was removed in gnolang#1297 in favor of `gno transpile -gobuild`.
harry-hov added a commit that referenced this pull request Apr 24, 2024
`gno build` was removed in #1297 in
favor of `gno transpile -gobuild`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 🤖 gnovm Issues or PRs gnovm related
Projects
Status: 🌟 Wanted for Launch
Archived in project
Development

Successfully merging this pull request may close these issues.

Refinement of gno precompile and build commands.
2 participants