Skip to content

Fetch commit messages for commits on GitHub#337

Merged
tmcgilchrist merged 1 commit intoocurrent:masterfrom
punchagan:capture-commit-message
May 10, 2022
Merged

Fetch commit messages for commits on GitHub#337
tmcgilchrist merged 1 commit intoocurrent:masterfrom
punchagan:capture-commit-message

Conversation

@punchagan
Copy link
Copy Markdown
Contributor

It is useful to fetch the commit message when fetching commits from GitHub. Current bench, for instance, could make the UI friendlier by displaying the commit message in the UI. ocurrent/current-bench#356

@punchagan
Copy link
Copy Markdown
Contributor Author

@tmcgilchrist could you take a look at this PR, please?

Copy link
Copy Markdown
Member

@tmcgilchrist tmcgilchrist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good thanks @punchagan

@punchagan punchagan force-pushed the capture-commit-message branch from d0600e0 to 3f8e63b Compare May 9, 2022 09:38
@punchagan
Copy link
Copy Markdown
Contributor Author

Thanks for taking a look, @tmcgilchrist! The build is green now. Could this be merged?

@tmcgilchrist tmcgilchrist merged commit f43ba44 into ocurrent:master May 10, 2022
@punchagan punchagan deleted the capture-commit-message branch May 10, 2022 04:15
tmcgilchrist added a commit to tmcgilchrist/opam-repository that referenced this pull request Jun 2, 2022
…, current_github, current_git, current_examples, current_docker and current (0.6.1)

CHANGES:

Web UI:

- UI restyle, embed files using ocaml-crunch. (@ewanmellor, @MisterDA, ocurrent/ocurrent#315)

- Automatically refresh some pipeline pages. (@MisterDA, ocurrent/ocurrent#227)

- Allow to import and export using CSV the log rules. (@MisterDA, ocurrent/ocurrent#327)

- Log matcher takes the pattern with the highest score (@kit-ty-kate, ocurrent/ocurrent#335)

API:

- GitHub: Add pp_short for commit and fix url link for GH commits. (@tmcgilchrist, ocurrent/ocurrent#347)

- GitHub: Fetch commit messages for commits on GitHub (@punchagan, ocurrent/ocurrent#337)

- GitHub: Add Current_github.Api.cmdliner_opt. (@dra27, ocurrent/ocurrent#338)

- GitHub: Add extra PR information to Ref.t (@tmcgilchrist @TheLortex, ocurrent/ocurrent#336)

Plugins:

- GitLab: support GitLab clone fork and fetch MR branch. (@MisterDA, ocurrent/ocurrent#346)

- Git: Fix handling of git repositories with submodules. (@talex5, ocurrent/ocurrent#345)

- GitLab: Fix ref filtering bug for GitLab (@tmcgilchrist, ocurrent/ocurrent#332)

Other:

- Add missing dependencies on Unix (@dra27, ocurrent/ocurrent#331)
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