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

Update to Remix 1.19.0 #1131

Merged
merged 2 commits into from
Jul 19, 2023
Merged

Update to Remix 1.19.0 #1131

merged 2 commits into from
Jul 19, 2023

Conversation

lpsinger
Copy link
Member

Remix 1.19.0 is in pre-releases right now, but the DX is so much nicer that folks might want to try it early!

@lpsinger lpsinger added the dependencies Pull requests that update a dependency file label Jul 18, 2023
@lpsinger
Copy link
Member Author

Remix 1.19.0 is out! This is now ready for review.

@lpsinger lpsinger requested review from dakota002 and Courey July 19, 2023 18:43
lpsinger added a commit to lpsinger/gcn.nasa.gov that referenced this pull request Jul 19, 2023
lpsinger added a commit to lpsinger/gcn.nasa.gov that referenced this pull request Jul 19, 2023
Fix the following eslint warnings revealed by nasa-gcn#1131:

```
app/components/NestedCheckboxes.tsx
   69:7   warning  Elements with the ARIA role "treeitem" must have the following attributes defined: aria-selected  jsx-a11y/role-has-required-aria-props
  111:15  warning  Elements with the ARIA role "treeitem" must have the following attributes defined: aria-selected  jsx-a11y/role-has-required-aria-props
```
@lpsinger lpsinger force-pushed the remix-1.19.0 branch 2 times, most recently from 4fb7d33 to 220bb3c Compare July 19, 2023 20:43
@codecov
Copy link

codecov bot commented Jul 19, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.10% ⚠️

Comparison is base (a987945) 5.83% compared to head (f163e00) 5.74%.

Additional details and impacted files
@@           Coverage Diff            @@
##            main   #1131      +/-   ##
========================================
- Coverage   5.83%   5.74%   -0.10%     
========================================
  Files         95      95              
  Lines       2209    2244      +35     
  Branches     270     273       +3     
========================================
  Hits         129     129              
- Misses      2078    2113      +35     
  Partials       2       2              
Files Changed Coverage Δ
app/components/NavStepIndicator.tsx 0.00% <0.00%> (ø)
app/entry.server.tsx 0.00% <0.00%> (ø)
app/lib/remix.ts 0.00% <0.00%> (ø)
app/root.tsx 0.00% <0.00%> (ø)
app/routes/user/email/index.tsx 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lpsinger lpsinger merged commit edaa06f into nasa-gcn:main Jul 19, 2023
@lpsinger lpsinger deleted the remix-1.19.0 branch July 19, 2023 21:20
lpsinger added a commit to lpsinger/gcn.nasa.gov that referenced this pull request Jul 19, 2023
I failed to update this when rebasing nasa-gcn#1131.
lpsinger added a commit that referenced this pull request Jul 19, 2023
I failed to update this when rebasing #1131.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants