-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat(config): add configuration parser and bundler #2200
Conversation
blacha
commented
May 17, 2022
- feat(config-cli): import configuration cli from linz/basemaps-config
- fix: include bin/
- feat: load screenshot config from file
- feat: add name to log lines
This pull request introduces 1 alert when merging 4f56419 into f962fe2 - view on LGTM.com new alerts:
|
4f56419
to
e4a20aa
Compare
This pull request introduces 1 alert when merging e4a20aa into 4694e29 - view on LGTM.com new alerts:
|
This pull request introduces 2 alerts when merging d4b3f62 into 4694e29 - view on LGTM.com new alerts:
|
e28ec4d
to
ac9a570
Compare
const id = Config.prefix(ConfigPrefix.Imagery, guessIdFromUri(uri)); | ||
|
||
const fileList = await fsa.toArray(fsa.list(uri)); | ||
const tiffFiles = fileList.filter((f) => f.endsWith('.tiff')); |
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.
Do we consider imagery end with .tif
?
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.
these our COGs so we decide what they end with