Skip to content

Conversation

@seanogdev
Copy link
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

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
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 😅


---

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

Choose a reason for hiding this comment

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

Copy link
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
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
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
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.

3 participants