Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Upload .well-known files using default ignore override #1631

Conversation

ObsidianMinor
Copy link
Contributor

@ObsidianMinor ObsidianMinor commented Nov 1, 2020

This removes all standard filters and replaces the faulty regex hack with a new default override that is used on top of any specified excludes.

Also this is getting more complex than it should be. Can we just use .gitignore or .ignore files on top of include/exclude and get rid of default "removed items" like node_modules and hidden files? Maybe just add a warning or add an option to preview a publish to see what would be uploaded?

@ObsidianMinor ObsidianMinor requested a review from a team as a code owner November 1, 2020 06:09
@ObsidianMinor ObsidianMinor force-pushed the fix/now-you-have-two-problems branch from e734ce4 to 57d6cbd Compare November 1, 2020 06:10
@ObsidianMinor ObsidianMinor force-pushed the fix/now-you-have-two-problems branch from 57d6cbd to ecec43c Compare November 1, 2020 06:14
@nataliescottdavidson
Copy link
Contributor

@ObsidianMinor, thanks for fixing this. Re .gitignore, docs for the ignore crate claim it respects that by default. I haven't tested to make sure that's the cause.

Adding a --dry-run option to publish is a great idea. I'll make an issue.

@nataliescottdavidson nataliescottdavidson merged commit da72960 into cloudflare:master Nov 2, 2020
nataliescottdavidson added a commit that referenced this pull request Nov 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants