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: bump react-pdf from 7 to 9 #2356

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

chore: bump react-pdf from 7 to 9 #2356

wants to merge 3 commits into from

Conversation

PiTrem
Copy link
Member

@PiTrem PiTrem commented Feb 27, 2025

  • fix post compilation issue for canva
  • webpack config alias for jsx runtime

@PiTrem PiTrem marked this pull request as draft February 27, 2025 21:26
Copy link

LCOV of commit 8c2dcc2 during Continuous Integration #4539

Summary coverage rate:
  lines......: 67.0% (15492 of 23122 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

PiTrem added a commit that referenced this pull request Mar 11, 2025
Squashed commit of the following:

commit b6faa3024ccd14d9c9d0b8758e607a8d48642156
Author: PiTrem <pierre.tremouilhac@kit.edu>
Date:   Thu Feb 27 19:44:41 2025 +0100

    wip_react-pdf

commit 8c2dcc2
Author: PiTrem <pierre.tremouilhac@kit.edu>
Date:   Wed Feb 26 12:46:44 2025 +0100

    chore: bump react-pdf from 7 to 9

    webpack config alias for jsx runtime
PiTrem added a commit that referenced this pull request Mar 12, 2025
Squashed commit of the following:

commit b6faa3024ccd14d9c9d0b8758e607a8d48642156
Author: PiTrem <pierre.tremouilhac@kit.edu>
Date:   Thu Feb 27 19:44:41 2025 +0100

    wip_react-pdf

commit 8c2dcc2
Author: PiTrem <pierre.tremouilhac@kit.edu>
Date:   Wed Feb 26 12:46:44 2025 +0100

    chore: bump react-pdf from 7 to 9

    webpack config alias for jsx runtime
@@ -0,0 +1,14 @@
class CspReportsController < ApplicationController

Choose a reason for hiding this comment

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

Style/FrozenStringLiteralComment: Missing frozen string literal comment.

head :ok
end
end

Choose a reason for hiding this comment

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

Layout/TrailingEmptyLines: 1 trailing blank lines detected.

# policy.object_src :none
# policy.script_src :self, :https
# policy.style_src :self, :https

Choose a reason for hiding this comment

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

Layout/TrailingWhitespace: Trailing whitespace detected.

# policy.style_src :self, :https

# policy.default_src :self, :https :unsafe_inline, :unsafe_eval

Choose a reason for hiding this comment

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

Layout/TrailingWhitespace: Trailing whitespace detected.

@@ -2,3 +2,4 @@

# Add new mime types for use in respond_to blocks:
# Mime::Type.register "text/richtext", :rtf
Mime::Type.register "application/javascript", :mjs

Choose a reason for hiding this comment

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

Style/StringLiterals: Prefer single-quoted strings when you don't need string interpolation or special symbols.

@@ -2,6 +2,7 @@
#
Rails.application.routes.draw do
post '/graphql', to: 'graphql#execute' unless Rails.env.production?
post "/csp-violation-report", to: "csp_reports#create"

Choose a reason for hiding this comment

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

Style/StringLiterals: Prefer single-quoted strings when you don't need string interpolation or special symbols.

Copy link

LCOV of commit 8d963af during Continuous Integration #4641

Summary coverage rate:
  lines......: 66.9% (15495 of 23146 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

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.

1 participant