Skip to content

Conversation

@quarckster
Copy link
Contributor

@quarckster quarckster commented Sep 24, 2019

What: Added OUIA props to Tabs as per #2425

Additional issues: tests failed with these errors and I cannot figure out what I should do next:

@quarckster
Copy link
Contributor Author

@redallen
Copy link
Contributor

Same deal as with Select in #2996 .

@patternfly-build
Copy link
Collaborator

PatternFly-React preview: https://patternfly-react-pr-2995.surge.sh

Copy link
Contributor

@tlabaj tlabaj left a comment

Choose a reason for hiding this comment

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

This one has some failing snapshot test. Can you please update them.

@quarckster quarckster changed the title [WIP] feat(Chip): Added OUIA compatibility to Chip component feat(Chip): Added OUIA compatibility to Chip component Sep 27, 2019
@quarckster
Copy link
Contributor Author

@tlabaj fixed

@karelhala karelhala added the ouia label Oct 1, 2019
karelhala
karelhala previously approved these changes Oct 1, 2019
tlabaj
tlabaj previously approved these changes Oct 3, 2019
Copy link
Contributor

@tlabaj tlabaj left a comment

Choose a reason for hiding this comment

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

can you please resolve merge conflicts so we can merge this one. Thanks

@tlabaj tlabaj self-assigned this Oct 3, 2019
dlabrecq
dlabrecq previously approved these changes Oct 4, 2019
redallen
redallen previously approved these changes Oct 4, 2019
@codecov-io
Copy link

codecov-io commented Oct 7, 2019

Codecov Report

Merging #2995 into master will increase coverage by <.01%.
The diff coverage is 65.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2995      +/-   ##
==========================================
+ Coverage   69.02%   69.02%   +<.01%     
==========================================
  Files         858      858              
  Lines       23413    23430      +17     
  Branches     1853     1859       +6     
==========================================
+ Hits        16160    16172      +12     
- Misses       6333     6337       +4     
- Partials      920      921       +1
Flag Coverage Δ
#misc 95.45% <ø> (ø) ⬆️
#patternfly3 69.22% <ø> (ø) ⬆️
#patternfly4 68.09% <65.21%> (ø) ⬆️
Impacted Files Coverage Δ
...fly-4/react-core/src/components/ChipGroup/Chip.tsx 78.84% <65.21%> (-4.02%) ⬇️

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 650c823...3bccc6e. Read the comment docs.

@redallen redallen merged commit bd358b6 into patternfly:master Oct 7, 2019
@patternfly-build
Copy link
Collaborator

Your changes have been released in:

  • @patternfly/react-core@3.113.0
  • @patternfly/react-docs@4.14.11
  • @patternfly/react-inline-edit-extension@2.11.79
  • demo-app-ts@3.6.18
  • @patternfly/react-table@2.22.28
  • @patternfly/react-topology@2.8.73
  • @patternfly/react-virtualized-extension@1.2.63

Thanks for your contribution! 🎉

@quarckster quarckster deleted the chipgroup_ouia branch October 7, 2019 13:22
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.

7 participants