Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Readme processing #419

Merged
merged 15 commits into from
Sep 28, 2022
Merged

Readme processing #419

merged 15 commits into from
Sep 28, 2022

Conversation

@AbdulrhmnGhanem AbdulrhmnGhanem temporarily deployed to staging September 22, 2022 09:33 Inactive
@AbdulrhmnGhanem AbdulrhmnGhanem temporarily deployed to staging September 22, 2022 11:29 Inactive
@AbdulrhmnGhanem AbdulrhmnGhanem added this to the alpha-2 milestone Sep 22, 2022
kasbah and others added 14 commits September 26, 2022 14:31
(cherry picked from commit 55ef041)
(cherry picked from commit e812496)

Waste a lot of time on test for readme images

(cherry picked from commit 1338747)
- Some projects use html in the readme, to fix the links in this case
  we will end up using cheerio again.
- `remark` on the other hand supports overriding html by a simple plugin
  `urlTansformer`.
- We were relying in a legacy behavior on gitea to redirect `/raw/`
  requets to the default branch.
- But if there is a branch with the same name of the file in
  `/raw/{file_path...}` it will redirect to the branch.
- This commit uses the default branch to avoid accidental redirects.
@AbdulrhmnGhanem AbdulrhmnGhanem temporarily deployed to staging September 27, 2022 11:40 Inactive
@AbdulrhmnGhanem AbdulrhmnGhanem temporarily deployed to staging September 27, 2022 12:05 Inactive
@kasbah kasbah merged commit e58dd6b into master Sep 28, 2022
@AbdulrhmnGhanem AbdulrhmnGhanem deleted the readme-processing branch July 22, 2023 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment