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

Support .yml YAML files in ember-intl functionality #384

Merged
merged 1 commit into from
Apr 26, 2022
Merged

Support .yml YAML files in ember-intl functionality #384

merged 1 commit into from
Apr 26, 2022

Conversation

patocallaghan
Copy link

@patocallaghan patocallaghan commented Apr 26, 2022

Follow up to lifeart/els-intl-addon#5 (comment)

Issue: #383

While the "official" file extension for YAML is .yaml many resources online still use .yml as the file extension of YAML. The application I work in has been using .yml for years and we'd love to use this addon without having to rename the files.

Ember Intl itself supports both the .yaml and .yml extension.

@codecov
Copy link

codecov bot commented Apr 26, 2022

Codecov Report

Merging #384 (349f774) into component-context-info-origin (5f3e91d) will decrease coverage by 0.04%.
The diff coverage is 90.00%.

@@                        Coverage Diff                        @@
##           component-context-info-origin     #384      +/-   ##
=================================================================
- Coverage                          72.40%   72.35%   -0.05%     
=================================================================
  Files                                 57       57              
  Lines                               4871     4884      +13     
  Branches                             946      949       +3     
=================================================================
+ Hits                                3527     3534       +7     
- Misses                              1344     1350       +6     
Impacted Files Coverage Δ
src/utils/usages-api.ts 56.57% <50.00%> (ø)
src/glimmer-utils.ts 52.50% <92.30%> (+2.50%) ⬆️
src/builtin-addons/core/intl-utils.ts 96.07% <100.00%> (+0.03%) ⬆️
...uiltin-addons/core/template-completion-provider.ts 93.02% <100.00%> (ø)
src/utils/yield-context-extractor.ts 55.20% <0.00%> (-4.00%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 253e5a6...349f774. Read the comment docs.

@lifeart lifeart merged commit e150ba7 into lifeart:component-context-info-origin Apr 26, 2022
lifeart pushed a commit that referenced this pull request Apr 26, 2022
# [2.24.0](v2.23.0...v2.24.0) (2022-04-26)

### Features

* Support .yml YAML files ([#384](#384)) ([e150ba7](e150ba7))
@lifeart
Copy link
Owner

lifeart commented Apr 26, 2022

@patocallaghan, 3.0.29 Stable Ember Language Server published

@patocallaghan
Copy link
Author

Thanks @lifeart 🎉 How soon does it take for this to be "published"? Should I reload my VSCode to pick up the new changes? (Sorry if these are silly questions)

@lifeart
Copy link
Owner

lifeart commented Apr 26, 2022

Should take about 5-15 minutes to be available in VSCode store, just search for it, and if it's installed - auto-update will be downloaded, restart is required

@patocallaghan
Copy link
Author

Thanks!!

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.

None yet

2 participants