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

[FIX] Use wikipedia for TIFF URL, adobe's page is 404ing now #1007

Merged
merged 1 commit into from
Feb 10, 2022

Conversation

yarikoptic
Copy link
Collaborator

Attn was brought up in #1006 (comment)
by @TheChymera

I think wikipedia would be the most reliable and informative source on TIFF format.

Attn @bids-standard/bep031

@yarikoptic
Copy link
Collaborator Author

IIRC 3 approvals is "enough" and don't remember if need to wait for someone to merge. Since change is trivial and could green up other PRs, I will use my super power of pressing a green button. Thanks everyone for speedy reaction!

@yarikoptic yarikoptic merged commit ba4923c into bids-standard:master Feb 10, 2022
@sappelhoff
Copy link
Member

sappelhoff commented Feb 10, 2022

just fyi: technically, the opening of a release PR (as was recently done by @franklin-feingold : #1004) "freezes" the spec and no PRs (except critical bugfixes) should be merged until the release PR is merged (after 5 days) and the release completed. That's to make any potential final reviews easier and to avoid confusion.

I just rebased the release PR on the merged changes from here and force-pushed, so it should be fine.

EDIT: Also, any PRs that are merged now (and not followed by a rebased+force-push on the release branch) will not show up on the changelog, due to the way we set things up

echo "Release PR - do nothing"

@sappelhoff sappelhoff changed the title BF: Use wikipedia for TIFF URL, adobe's page is 404ing now [FIX] Use wikipedia for TIFF URL, adobe's page is 404ing now Feb 10, 2022
@yarikoptic yarikoptic deleted the bf-tiff branch April 30, 2024 23:57
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.

4 participants