Skip to content

Conversation

@mhamas
Copy link
Contributor

@mhamas mhamas commented Oct 27, 2020

No description provided.

@mhamas mhamas self-assigned this Oct 29, 2020
// Prior to that, all default branches were 'master', hence we default to that as a backup.
const branchName = parsedRepoUrl.hash || gitBranchName || 'master';

// Images in markdown have syntax ![alt text](image url)
Copy link
Member

Choose a reason for hiding this comment

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

There could be also includes as HTML (for example when embedded in HTML table) - <img src=... />

@mhamas
Copy link
Contributor Author

mhamas commented Oct 30, 2020

@mtrunkat I've addressed the comments and verified locally that this works fine.

@mtrunkat mtrunkat self-requested a review November 3, 2020 14:09
@mhamas mhamas merged commit 5abdddf into master Nov 3, 2020
@mhamas mhamas deleted the feature/convert-relative-image-paths-to-absolute-in-repo-readme branch November 3, 2020 15:05
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