-
-
Notifications
You must be signed in to change notification settings - Fork 135
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
V6 #216
Merged
Merged
V6 #216
Changes from 1 commit
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
5210758
Fail CI if package.json and yarn lock are out of sync (#186)
koenpunt 0a84b90
Warn against filename-unfriendly chars in moduleName (#187)
d49dad0
Remove support for Node 4 and 6 (#190)
0783a34
update packages
chrisblossom a7c760e
fix npm config to correctly remove package-lock.json
chrisblossom e8a16e9
enable test undefined checks
chrisblossom 1b452bc
set prettier trailingComma to all
chrisblossom 6003890
set node/no-unsupported-features to first node 8 LTS release when new…
chrisblossom 6da6871
use prettier for all supported file types (except markdown) (#193)
chrisblossom 26a961a
add eslint-plugin-jest (#196)
chrisblossom 128e42c
use babel to compile source
chrisblossom 5560b29
remove unused coverage script
chrisblossom b68a566
add pre-push git hook, and consolidate all checks into check:all script
chrisblossom 1d4b35b
eslint: remove plugin-node from src enable module parsing, and remove…
chrisblossom 8f7d24a
convert common-js to es modules named exports
chrisblossom 53d2d40
move flow-typed/defs.js to src/types.js and remove unused types
chrisblossom 748a03c
migrate build system to typescript
chrisblossom f9c6880
remove flow comments
chrisblossom e2e5dd9
test files to typescript
chrisblossom ed37ddd
source files to typescript
chrisblossom cb058a1
run prettier on all files
chrisblossom 0ea62ce
rename all files to .ts and update changelog
chrisblossom 47d8951
Typescript code quality tools (#202)
chrisblossom 0e10c02
export Options and combine multiple Options definitions
chrisblossom 709ca57
export Transform and add correctly type it can be a Promise (async only)
chrisblossom f18cf0c
fix cacheWrapper to cache the result instead of a Promise, merge mult…
chrisblossom 0d7b647
export LoaderSync and LoaderAsync
chrisblossom e98bd6a
correctly type LoadedFileContent
chrisblossom c6ef21d
split loadFileContent into async / sync separate functions
chrisblossom 5a7053e
use default function parameters
chrisblossom dc17101
use object spread
chrisblossom 2dd9157
export getDirectorySync
chrisblossom 9f9dda9
export readFileSync
chrisblossom 76547e4
update import-fresh to use built-in types
chrisblossom 17f743f
replace is-directory with path-type
chrisblossom b9649ec
remove unused global.d.ts
chrisblossom eec912f
use builtin Object.prototype.hasOwnProperty
chrisblossom 3ccdea8
group async and sync versions of functions, async first
chrisblossom 4933f69
use async await where possible
chrisblossom 3f520f4
convert tests that throw to .toThrow syntax
chrisblossom 9c9adc1
Prevent pushing and publishing with unformatted files
chrisblossom 318d638
Breaking: remove `cosmiconfig.loaders` and add named export `defaultL…
chrisblossom 67d41b0
Tweak readme
69696dd
split create explorer async and sync methods into different files
chrisblossom 2a270c0
createExplorer and createExplorerSync
chrisblossom 24bc87e
remove excess getLoaderEntryForFile functions
chrisblossom 91209d2
update readme and changelog
chrisblossom 7e8e11a
Minor v6 test revisions (#217)
ad63f7f
rename createExplorer*
chrisblossom b89752e
Move createExplorer* to index.ts
chrisblossom a7bc28e
remove outdated line in readme
chrisblossom 26b65bb
export actual defaultLoaders object
chrisblossom 7a0f0e6
freeze defaultLoaders so it cannot be mutated causing unexpected results
chrisblossom 83ff4b1
update changelog
chrisblossom 60337c9
replace js-yaml with yaml
chrisblossom 6935c0f
update all packages and ensure prefix is correctly set for future npm…
chrisblossom 1e1ae77
refactor test/index.test.ts to remove jest inline snapshot hacks
chrisblossom fad0ebc
remove yarn.lock
chrisblossom 080256a
Revert "remove yarn.lock"
chrisblossom 358104c
Include absolute file path to yaml parse error
chrisblossom db8f2cd
windows del fix
chrisblossom c8c8817
remove yarn.lock
chrisblossom 08ebcd5
Lazy load all default loaders
chrisblossom ed83eaa
Prepare 6.0.0
b7b2ae6
Fix typo
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am unsure how to fix this test. I don't understand the point of the test, don't we know that by requiring it?. I personally think we should just remove it and move on.