Gregor Martynus

If source (`current_name`) branch does not exist, or if target (`name`) branch exist, throw a helpful error

Look for all `<current_name>` occurences in the code and replace them with the new branch name. Create a pull request for review

2

There is no simple way to avoid false positives, but there are several places where the current branch name needs to be replaced: - README badges - CI configurations: GitHub...

As semantic-release supports publishing from all these branches

Run `dependency-check` in Test Action Workflow

1

I just found out by chance that `@octokit/auth-oauth-app` was using `universal-user-agent` without having it listed as dependency. That caused a problem with loading the package from https://cdn.skypack.dev/@octokit/[email protected]. I did a...

e.g. plugins should have `compose*` methods besides the plugin export, compare `composePaginateRest` from [`@octokit/plugin-paginate-rest`](https://github.com/octokit/plugin-paginate-rest.js#usage)

Remove `// @ts-ignore` statements

2

I had to rush the transition to TypeScript a little bit. Instead of setting the correct types everywhere, I've added a few `// @ts-ignore` statements: https://github.com/octokit/plugin-retry.js/search?q=%22%2F%2F+%40ts-ignore%22&unscoped_q=%22%2F%2F+%40ts-ignore%22 I'd love a pull...

💡On 500 server errors, check GitHub Status

1

Just an idea for later :tm: We can check https://www.githubstatus.com/api/v2/status.json if the request went to https://api.github.com and the server responded with 500, that way we could provide better error messages...

Listen to "synchronize" pull_request action

3

Hey @hiimbex :wave: right now the app only listens to the "open" action https://github.com/hiimbex/markdownify/blob/f83012eafcb38b12de61e55e18305359dccb92fc/index.js#L2 After installing the app, I’d expect that the app would also listen to the "synchronize" action...

Do not list removed Markdown files

2

When I created https://github.com/octokit/rest.js/pull/1085, mardownify listed `lib/endpoint/README.md` which was removed compared to `master`. I since changed the base to `next` so it no longer is edited in the PR