Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix GitGitGadget's open PRs -> cover letter Message-ID map#70

Merged
dscho merged 1 commit intogitgitgadget:mastergitgitgadget/gitgitgadget:masterfrom
dscho:open-prsdscho/gitgitgadget:open-prsCopy head branch name to clipboard
Jan 17, 2019
Merged

Fix GitGitGadget's open PRs -> cover letter Message-ID map#70
dscho merged 1 commit intogitgitgadget:mastergitgitgadget/gitgitgadget:masterfrom
dscho:open-prsdscho/gitgitgadget:open-prsCopy head branch name to clipboard

Conversation

@dscho
Copy link
Member

@dscho dscho commented Jan 14, 2019

By mistake, the Message-ID was left empty.

By mistake, the Message-ID was left empty.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho requested a review from derrickstolee January 14, 2019 13:27
@dscho
Copy link
Member Author

dscho commented Jan 15, 2019

To clarify: the current GitGitGadget options contains:

    "openPRs": {
        "https://github.com/gitgitgadget/git/pull/100": "pull.100.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/101": "",
        "https://github.com/gitgitgadget/git/pull/103": "",
        "https://github.com/gitgitgadget/git/pull/104": "",
        "https://github.com/gitgitgadget/git/pull/105": "",
        "https://github.com/gitgitgadget/git/pull/106": "",
        "https://github.com/gitgitgadget/git/pull/107": "",
        "https://github.com/gitgitgadget/git/pull/15": "pull.15.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/24": "",
        "https://github.com/gitgitgadget/git/pull/29": "pull.29.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/31": "pull.31.v2.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/50": "pull.50.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/59": "pull.59.v2.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/66": "pull.66.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/82": "pull.82.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/83": "pull.83.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/87": "pull.87.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/89": "pull.89.v3.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/90": "pull.90.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/92": "",
        "https://github.com/gitgitgadget/git/pull/95": "pull.95.git.gitgitgadget@gmail.com",
        "https://github.com/gitgitgadget/git/pull/96": "pull.96.git.gitgitgadget@gmail.com"
    }

Clearly, some newer entries lack the Message-ID of the corresponding cover letter. This PR is intended to fix that for future submissions.

@dscho
Copy link
Member Author

dscho commented Jan 16, 2019

@derrickstolee could you have a look?

@dscho dscho merged commit 04c6016 into gitgitgadget:master Jan 17, 2019
@dscho dscho deleted the open-prs branch January 17, 2019 11:04
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.