Feature: Show head & base ref, cap markdown headings to h4 by seanogdev · Pull Request #42 · Teamwork/github-sync · GitHub
Skip to content

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

Merged
seanogdev merged 6 commits into
masterfrom
add_head_ref
Jul 22, 2021
Merged

Feature: Show head & base ref, cap markdown headings to h4#42
seanogdev merged 6 commits into
masterfrom
add_head_ref

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

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.

@Flexicon Flexicon left a comment

Copy link
Copy Markdown
Contributor

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

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