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

Losslessly Optimize Images #86

Closed
wants to merge 3 commits into from
Closed

Conversation

daknob
Copy link
Contributor

@daknob daknob commented Dec 3, 2015

Optimize all repository images using lossless compression (ImageOptim was used).

@ssuchanowski
Copy link

I think you went too far with ones with red background

@katzenbaer
Copy link

Can you change the title of Pull Requests? You have a typo: Optmize instead of Optimize.

@wadestuart
Copy link

Yeah, you will find that images that have strong 1 color channel prevalent areas tend to get pretty sever artifacts and color shift when doing this type of optimization.

@cwillmor cwillmor changed the title Losslessly Optmize Images Losslessly Optimize Images Dec 4, 2015
@tmandry
Copy link

tmandry commented Dec 4, 2015

Problem is, these are already part of the git history... overwriting them will only make the repo larger. For docs it makes sense if they are hosted as web pages, but not really for tests/utils.

@daknob
Copy link
Contributor Author

daknob commented Dec 4, 2015

Update: All changes have been reverted. Only Documentation images are now optimized since there is no quality loss.

Thanks for the feedback.

@alanmcgovern
Copy link

Committing something and immediately reverting it will bloat the repository, as was already mentioned. You should squash the commits into one so there's no unnecessary images added to history. Or just re-issue the PR using just the third commit, which is the only one that matters.

@daknob daknob closed this Dec 4, 2015
@daknob daknob deleted the image-optim branch December 4, 2015 13:37
dabelknap pushed a commit to dabelknap/swift that referenced this pull request Aug 2, 2018
dabelknap pushed a commit to dabelknap/swift that referenced this pull request Sep 17, 2018
slavapestov pushed a commit to slavapestov/swift that referenced this pull request Nov 27, 2018
…n_linux

eliminate mach_task_self from linux_stubs
slavapestov pushed a commit to slavapestov/swift that referenced this pull request Nov 27, 2018
…n_linux

eliminate mach_task_self from linux_stubs

Signed-off-by: Daniel A. Steffen <[email protected]>
freak4pc pushed a commit to freak4pc/swift that referenced this pull request Sep 28, 2022
DougGregor pushed a commit to DougGregor/swift that referenced this pull request Apr 28, 2024
motivation: dedup more across RPM spec and scripts

changes:
* organize shared content in the a "Shared" directory
* create several more ".inc" files that extract duplicate information across RPM spec, including the description and header
* add symlinks to shared metadata and script so that the spec can also run easily outide docker
* update docker compose scripts to use new directory structure
* update Dockerfile optimization to be more resiliant by rewrting an bare-bone optimnization spec
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.

6 participants