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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: remove version number from axe.d.ts #1499

Merged
merged 1 commit into from
Apr 17, 2019
Merged

Conversation

stephenmathieson
Copy link
Member

@stephenmathieson stephenmathieson commented Apr 16, 2019

This patch removes the version number from axe.d.ts. I think the original idea behind keeping it here was to update it every time we release, but this hasn't been happening lately.

Removing the version does not have any effect on TypeScript users; instead, it just removes an easy to break/forget part of our internal processes 馃槂

Reviewer checks

Required fields, to be filled out by PR reviewer(s)

  • Follows the commit message policy, appropriate for next version
  • Has documentation updated, a DU ticket, or requires no documentation change
  • Includes new tests, or was unnecessary
  • Code is reviewed for security by: @JKODU

This patch removes the version number from `axe.d.ts`. I think the original idea behind keeping it here was to update it every time we release, but this hasn't been happening lately.

Removing the version does not have any effect on TypeScript users; instead, it just removes an easy to break/forget part of our internal processes 馃槂
@stephenmathieson stephenmathieson requested a review from a team as a code owner April 16, 2019 20:45
@WilcoFiers WilcoFiers merged commit 6cf8b2f into develop Apr 17, 2019
@WilcoFiers WilcoFiers deleted the remove-tsdefs-version branch April 17, 2019 10:10
stephenmathieson added a commit to mohanraj-r/axe-core that referenced this pull request May 10, 2019
* develop: (50 commits)
  docs: add jsdom example and tests (dequelabs#1530)
  fix(aria-valid-attr-value): allow aria-owns to pass when element is not in the DOM (dequelabs#1526)
  fix(isSkipLink): cache first page link (dequelabs#1525)
  chore: update Babel dependencies (dequelabs#1527)
  chore(package): Update karma to version 4.1.0 (dequelabs#1528)
  feat: Improve perf of axe.run [WWD-1821] (dequelabs#1503)
  fix(prettier): ignore generated api doc files (dequelabs#1522)
  fix(skip-link,region): Allow multiple skiplinks at page top (dequelabs#1496)
  fix(raw-reporter): do not output `DqElement`s (dequelabs#1513)
  fix: Scroll state had top and left properties flipped (dequelabs#1469)
  refactor: commons.color.getBackgroundColor method (dequelabs#1451)
  fix(aria-valid-attr-value): allow aria-controls to pass when element is not in the DOM
  chore: Update husky to the latest version 馃殌 (dequelabs#1514)
  style: format HTML files with Prettier (dequelabs#1508)
  test: Fix invalid test html (dequelabs#1502)
  feat(rule): Inline text spacing must be adjustable with custom stylesheets (dequelabs#1446)
  chore: Remove version number from axe.d.ts (dequelabs#1499)
  chore: Update make-dir to the latest version 馃殌 (dequelabs#1465)
  fix: Exclude  iframe for html-has-lang rule (Issue 1424) (dequelabs#1430)
  feat(utils): add support for complex CSS selectors (dequelabs#1494)
  ...
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

3 participants