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

chore(release): 4.8.4 #4328

Merged
merged 7 commits into from Feb 7, 2024
Merged

chore(release): 4.8.4 #4328

merged 7 commits into from Feb 7, 2024

Conversation

straker
Copy link
Contributor

@straker straker commented Feb 7, 2024

No description provided.

WilcoFiers and others added 5 commits February 5, 2024 10:35
Axe-core needs to include the copyright and licenses of third-party
software.

I also looked at preserving the copyright statements from third-party
libraries where they are embedded. Unfortunately that will require us
addressing the webpack issue due to which we pinned our ESBuild version.
#4305

---------

Co-authored-by: Dan Bjorge <dan.bjorge@deque.com>
…de types (#4317)

Updates VirtualNode's `props` initialization path to avoid reading
properties that we know based on `nodeType` won't be present anyway.
This should mitigate #4316 by avoiding reading the problematic `value`
prop present on certain text nodes.

I also cleaned up some missing test coverage in the impacted code.

Closes: #4316
…elector (#4303)

Ignore the "xmlns" attribute when generating a selector, since the
selector spec does not support namespaces.

Closes: #4302
* fix(d.ts): RawNodesResult issues

* Type tests

* More

* empty

* Fixed feedback
Co-authored-by: Steven Lambert <2433219+straker@users.noreply.github.com>
@straker straker requested a review from a team as a code owner February 7, 2024 16:34
@straker straker changed the base branch from develop to master February 7, 2024 16:34
@@ -13,6 +13,7 @@
},
"devDependencies": {
"axe-core": "^4.6.2",
"chai": "^4.3.7",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We needed to get the test/examples passing but trying to cherry-pick #4296 failed with conflicts so we'll make the change manually and deal with the merge conflict when it merges back into develop

Copy link
Contributor

@WilcoFiers WilcoFiers left a comment

Choose a reason for hiding this comment

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

@WilcoFiers WilcoFiers merged commit 9ba9d05 into master Feb 7, 2024
18 checks passed
@WilcoFiers WilcoFiers deleted the release-4.8.4 branch February 7, 2024 17:44
@dylanb
Copy link
Contributor

dylanb commented Feb 7, 2024

reviewed

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

5 participants