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

Issue Fix: #12 Add a link to the video overview to the README #16

Closed
wants to merge 1 commit into from

Conversation

AXE-00
Copy link

@AXE-00 AXE-00 commented Dec 6, 2023

No description provided.

@Lextuga007 Lextuga007 linked an issue Dec 7, 2023 that may be closed by this pull request
@Lextuga007 Lextuga007 changed the title Issue Fix: #12 Issue Fix: #12 Add a link to the video overview to the README Dec 7, 2023
@Lextuga007 Lextuga007 self-assigned this Dec 7, 2023
@ThomUK
Copy link
Collaborator

ThomUK commented Dec 7, 2023

Thank you for your contribution. We are reviewing.

@ThomUK ThomUK closed this Dec 7, 2023
@Lextuga007
Copy link
Member

Lextuga007 commented Dec 7, 2023

Just for clarity we are unsure about a png in the README as that is quite big and can bloat a repository as it's in the commit history. Where it's absolutely necessary, like in training slides we've had help going through those commit histories previously to reduce the size.

This could be a good opportunity to try out removing the png from the PR and history using https://rtyley.github.io/bfg-repo-cleaner/ as recommended by GitHub and will require Java being on the computer (and maybe admin rights depending on what organisation you work in and the security of networks).

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.

Add a link to the video overview to the README
3 participants