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

clean sugar github #31

Merged
merged 11 commits into from
Nov 24, 2023
Merged

clean sugar github #31

merged 11 commits into from
Nov 24, 2023

Conversation

amtoine
Copy link
Owner

@amtoine amtoine commented Oct 20, 2023

related to

description

this cleans up the gh.nu module into github.nu and will address one task in #106

new commands

  • gm gh query-api: this is the biggest one which does all the API heavy lifting
  • gm gh query-releases
  • gm gh query-user
  • gm gh pr checkout inspired from old me pr

removed commands

  • all the old gh.nu module
  • the now unused fs/dir.nu module

tests

i don't think i'll add tests for that right now as the targets will be moving 🤔

@amtoine amtoine added the sugar Related to the `nu-git-manager-sugar` module label Nov 2, 2023
@amtoine amtoine added the enhancement New feature or request label Nov 23, 2023
@amtoine amtoine marked this pull request as ready for review November 23, 2023 18:42
@amtoine amtoine mentioned this pull request Nov 23, 2023
3 tasks
@amtoine
Copy link
Owner Author

amtoine commented Nov 23, 2023

after that PR, i'll add the new feature from nushell/nushell#11057 to simplify the --quiet, --no-gh and --no-paginate switches 😉

@amtoine amtoine changed the title GitHub API query clean sugar github Nov 23, 2023
@amtoine amtoine merged commit 5463513 into main Nov 24, 2023
4 checks passed
@amtoine amtoine deleted the github-api-query branch November 24, 2023 17:40
amtoine added a commit to amtoine/dotfiles that referenced this pull request Nov 24, 2023
amtoine added a commit that referenced this pull request Nov 24, 2023
related to
- #31

## description
i found two bugs when using `gm gh pr checkout`
- in `nu-git-manager`, i pressed escape in `input list` and there was no
early `return` => the command crashed on `$res.id`
- in `nushell`, my GitHub remote repo was incorrectly set to
`amtoine/nushell` => the command crashed because the PR table was empty

this PR fixes those two bugs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request sugar Related to the `nu-git-manager-sugar` module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant