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

fix(misc/list-gnophers): make script work when called from any directory #1308

Merged
merged 2 commits into from
Nov 6, 2023

Conversation

thehowl
Copy link
Member

@thehowl thehowl commented Oct 26, 2023

+ use a tmpdir for the csv file.

@moul

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
  • [x ] 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.

@thehowl thehowl requested a review from moul October 26, 2023 22:52
@thehowl thehowl requested a review from a team as a code owner October 26, 2023 22:52
@moul
Copy link
Member

moul commented Nov 6, 2023

thank you 👍

merging with just my review, because in misc/ and only for scripting, not for dev/production.

@moul moul merged commit ce7a7c6 into master Nov 6, 2023
6 of 7 checks passed
@moul moul deleted the dev/morgan/list-gnophers-any-dir branch November 6, 2023 17:57
gfanton pushed a commit to gfanton/gno that referenced this pull request Nov 9, 2023
…ory (gnolang#1308)

\+ use a tmpdir for the csv file.

@moul 

<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>
moul pushed a commit to moul/gno that referenced this pull request Nov 14, 2023
…ory (gnolang#1308)

\+ use a tmpdir for the csv file.

@moul 

<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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants