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 Merge pull request #656 to remove const enums usage. #681

Merged
merged 3 commits into from
Apr 3, 2020

Conversation

subgan82
Copy link
Contributor

@subgan82 subgan82 commented Apr 2, 2020

This reverts commit f3f856b, reversing
changes made to 8a2daac.

Based on this discussion #668 (comment)

#668 (comment)

Const Enums cannot be used by babel.

This reverts commit f3f856b, reversing
changes made to 8a2daac.
@subgan82 subgan82 changed the title Revert "Merge pull request #656 from maneetgoyal/symbol-enums" Revert Merge pull request #656 to remove const enums usage. Apr 2, 2020
@subgan82
Copy link
Contributor Author

subgan82 commented Apr 2, 2020

cc; @maneetgoyal @kumarGayu

@codecov
Copy link

codecov bot commented Apr 2, 2020

Codecov Report

Merging #681 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #681   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          107       106    -1     
  Lines         1616      1613    -3     
  Branches       286       286           
=========================================
- Hits          1616      1613    -3     
Impacted Files Coverage Δ
packages/arcgis-rest-types/src/index.ts

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 34f5037...41e1e7c. Read the comment docs.

@subgan82
Copy link
Contributor Author

subgan82 commented Apr 2, 2020

@tomwayson / @patrickarlt

All the changes introduced by this PR #656 has been reverted here.

Copy link
Contributor

@patrickarlt patrickarlt left a comment

Choose a reason for hiding this comment

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

Looks good to me. I'll let someone else (@tomwayson maybe?) review and deploy 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.

4 participants