Skip to content

fix: set Loc Line on Decl #2221

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jun 3, 2024
Merged

fix: set Loc Line on Decl #2221

merged 2 commits into from
Jun 3, 2024

Conversation

omarsy
Copy link
Member

@omarsy omarsy commented May 27, 2024

Closes #2220

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.

@github-actions github-actions bot added the 📦 🤖 gnovm Issues or PRs gnovm related label May 27, 2024
Copy link

codecov bot commented May 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.08%. Comparing base (e1b0dc0) to head (eaab6e1).
Report is 26 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2221      +/-   ##
==========================================
+ Coverage   49.07%   50.08%   +1.01%     
==========================================
  Files         576      577       +1     
  Lines       77821    78291     +470     
==========================================
+ Hits        38192    39215    +1023     
+ Misses      36534    35897     -637     
- Partials     3095     3179      +84     
Flag Coverage Δ
gnovm 45.11% <100.00%> (+2.91%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@omarsy omarsy changed the title fix: set Loc Line on Decls fix: set Loc Line on Decl May 29, 2024
Copy link
Member

@thehowl thehowl left a comment

Choose a reason for hiding this comment

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

👌

cc @mvertes to verify the changes on the debugger outputs.

@thehowl thehowl merged commit ed2835d into gnolang:master Jun 3, 2024
47 checks passed
@omarsy omarsy deleted the fix/2220 branch June 3, 2024 18:30
omarsy added a commit to TERITORI/gno that referenced this pull request Jun 3, 2024
Closes gnolang#2220
<!-- please provide a detailed description of the changes made in this
pull request. -->

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

- [ ] 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](https://gnoland.github.io/benchmarks), if any. More info
[here](https://github.com/gnolang/gno/blob/master/.benchmarks/README.md).
</details>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
📦 🤖 gnovm Issues or PRs gnovm related
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

We didn't set File line on ValueDeclr ImportDecl, TypeDecl
4 participants