Skip to content

Conversation

@Mashal-m
Copy link
Contributor

Browsers list has extended in package.json.

@openedx-webhooks openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Sep 21, 2022
@openedx-webhooks
Copy link

Thanks for the pull request, @Mashal-m! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

  • supporting documentation
  • Open edX discussion forum threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

⚠️ We can't start reviewing your pull request until you've submitted a signed contributor agreement or indicated your institutional affiliation. Please see the CONTRIBUTING file for more information. If you've signed an agreement in the past, you may need to re-sign. See The New Home of the Open edX Codebase for details.

@codecov
Copy link

codecov bot commented Sep 21, 2022

Codecov Report

Base: 82.91% // Head: 82.86% // Decreases project coverage by -0.04% ⚠️

Coverage data is based on head (3461dee) compared to base (4ff91d0).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #863      +/-   ##
==========================================
- Coverage   82.91%   82.86%   -0.05%     
==========================================
  Files         355      355              
  Lines        8674     8686      +12     
  Branches     1945     1948       +3     
==========================================
+ Hits         7192     7198       +6     
- Misses       1453     1459       +6     
  Partials       29       29              
Impacted Files Coverage Δ
...credit-management/LearnerCreditAllocationTable.jsx 21.73% <0.00%> (-4.93%) ⬇️
...components/learner-credit-management/data/utils.js 96.29% <0.00%> (+4.99%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Mashal-m Mashal-m closed this Oct 4, 2022
@Mashal-m Mashal-m reopened this Oct 4, 2022
@Mashal-m Mashal-m changed the title build:browserslist extended in package.json build: browserslist extended in package.json Oct 4, 2022
@Mashal-m Mashal-m closed this Oct 4, 2022
@Mashal-m Mashal-m reopened this Oct 4, 2022
@Mashal-m Mashal-m changed the title build: browserslist extended in package.json build: use shared browserslist configuration Oct 4, 2022
@Mashal-m Mashal-m closed this Oct 4, 2022
@Mashal-m Mashal-m reopened this Oct 4, 2022
@Mashal-m Mashal-m closed this Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

open-source-contribution PR author is not from Axim or 2U

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants