Skip to content

Remove gb wls empty regions#445

Open
tekbreak wants to merge 2 commits into
commenthol:masterfrom
tekbreak:remove_GB_wls_empty_regions
Open

Remove gb wls empty regions#445
tekbreak wants to merge 2 commits into
commenthol:masterfrom
tekbreak:remove_GB_wls_empty_regions

Conversation

@tekbreak

Copy link
Copy Markdown

Fix for empty object in GB - WLS regions.
Fix failing build when Json file is missing

Comment thread data/countries/GB.yaml
name:
en: Summer bank holiday
regions: {}
# regions:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread CONTRIBUTING.md
3. Create Json file and build the project:
```
npm run build
npm run yaml && npm run build

@KristjanESPERANTO KristjanESPERANTO Jun 29, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since 1ee2e82 the yaml command is already part of the build command, so we'd just add unnecessary overhead by running it twice.

@KristjanESPERANTO KristjanESPERANTO left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tekbreak Thanks for this PR! But I think it's not necessary anymore.

@commenthol I'd recommend closing this :)

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.

2 participants