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

Feature: Show head & base ref, cap markdown headings to h4#42

Merged
seanogdev merged 6 commits into
masterTeamwork/github-sync:masterfrom
add_head_refTeamwork/github-sync:add_head_refCopy head branch name to clipboard
Jul 22, 2021
Merged

Feature: Show head & base ref, cap markdown headings to h4#42
seanogdev merged 6 commits into
masterTeamwork/github-sync:masterfrom
add_head_refTeamwork/github-sync:add_head_refCopy head branch name to clipboard

Conversation

@seanogdev
Copy link
Copy Markdown
Contributor

Added the base and head branches. Found myself losing context sometimes & to avoid going between github & teamwork, I'd rather have the branch information in teamwork itself.

Also added the base ref, useful when working in a long standing feature branch

Finally, I've capped all headings to h4 , as the headings in teamwork's markdown are quite big

Comment thread src/github.sh Outdated

github::get_pr_body() {
jq --raw-output .pull_request.body "$GITHUB_EVENT_PATH"
jq --raw-output .pull_request.body "$GITHUB_EVENT_PATH" | sed -r 's/#{1,3}/####/'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

someone with more than cursory knowledge of sed might wanna take a look at this 😅

Comment thread src/teamwork.sh

---

${pr_body//###/####}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

@Flexicon Flexicon left a comment

Choose a reason for hiding this comment

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

It looks good to me, but I'm assuming you ran the bash scripts locally and made sure sed is doing what you want it to? 😅

@seanogdev seanogdev marked this pull request as draft July 22, 2021 11:08
@seanogdev
Copy link
Copy Markdown
Contributor Author

It looks good to me, but I'm assuming you ran the bash scripts locally and made sure sed is doing what you want it to? 😅

act/docker is acting up actually... 2 secs

@seanogdev
Copy link
Copy Markdown
Contributor Author

Yep so act has a bug right now, tested the event directly

image

@seanogdev seanogdev marked this pull request as ready for review July 22, 2021 11:43
@seanogdev seanogdev merged commit ce659ae into master Jul 22, 2021
@seanogdev seanogdev deleted the add_head_ref branch July 22, 2021 12:42
@Flexicon
Copy link
Copy Markdown
Contributor

Good enough for me! 👍

Yep so act has a bug right now, tested the event directly

image

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.