Skip to content

Conversation

@leandro-lucarella-sociomantic

This PR adds several new features:

  • Dry-run mode, to see what it will be done before actually doing it
  • Add support for OAuth authentication
  • Properly close copied issues when the original was closed
  • Add support to "move" issues (meaning, the original issue will be
    closed if it was opened, and adding a comment saying the issue was
    moved in any case).

Other small fixes are included too:

  • Load all the milestones, not only open ones
  • Add support for custom verbs to send_request(), this was necessary to
    update issues (close them)
  • Don't ask questions when not in a tty (this allows piping the program
    using xargs, for example, to get a list of issues to import)
  • Remove trailing white spaces

I know including many features in one PR might be a bit suboptimal,
given that all have to be accepted (or rejected) together, but splitting
it in several PR could be some work, so I would prefer to do it only if
necessary.

Only the POST requests are omitted, the rest of the work is performed
completely.
This allows the program to be called via a pipe, for example to get
a list of issues to copy from another command and then invoking the
import via | xargs.
This will enable doing other operations with the GitHub API, like marking
copied issues that were closed as closed.
Before a prefix was added to the issue title, now the issue is properly
closed.
When importing issues closed issues, we also probably want to import closed
milestones too, or at least don't break when issues associated with closed
milestones.
Moving issues means closing the original issue and adding a comment saying
the issue was moved (through a comment template).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant