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

Revert some unused code #260

Merged
merged 4 commits into from
Jan 26, 2022
Merged

Revert some unused code #260

merged 4 commits into from
Jan 26, 2022

Conversation

mattolson
Copy link
Contributor

Revert some changes over the past year that are unused or related to other reversions.

  • Revert addition of preprocess and addTemplates which are unused
  • Revert refactor that exposed setLanguage function which is unused outside of Translator class
  • Remove unused Translator.compileFormatterFunction
  • Revert changes to translation compilation error handling leftover from failed MessageFormat upgrade

These are not used anywhere and should be reverted as part of the
overall reversion related to GraalVM
Revert refactor that exposed setLanguage function
Revert changes to translation compilation error handling leftover from
failed MessageFormat upgrade
@mattolson mattolson requested a review from jairo-bc January 26, 2022 19:47
Copy link
Contributor

@jairo-bc jairo-bc left a comment

Choose a reason for hiding this comment

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

LGTM, one comment left

this.setLanguage(languages);
/**
* @private
* @type {Object.<string, string>}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you don't need those statements here anymore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm just reverting back to what it was - if we want to remove these, let's do it consistently for all of them in a separate change.

@mattolson mattolson merged commit b3e0440 into bigcommerce:master Jan 26, 2022
@mattolson mattolson deleted the STRF-9433 branch January 26, 2022 21:43
@mattolson mattolson mentioned this pull request Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants