Skip to content

PFDA-6991 add security.md file - #124

Closed
gitbisect wants to merge 16 commits into
masterfrom
PFDA-6991-add-security-md
Closed

PFDA-6991 add security.md file#124
gitbisect wants to merge 16 commits into
masterfrom
PFDA-6991-add-security-md

Conversation

@gitbisect

Copy link
Copy Markdown
Collaborator

🗒️ Description

https://jira.internal.dnanexus.com/browse/PFDA-6991

Please read the Jira ticket for more details.

📐 Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Feature change (modifies existing functionality, may affect user experience or output)
  • Breaking change (fix or feature that may cause existing functionality to not work as expected)

Risk analysis was not conducted yet.

jirkalagan and others added 16 commits May 28, 2026 12:54
Co-authored-by: Jiří Lagan <43816355+jirkalagan@users.noreply.github.com>
Co-authored-by: Michal Vojtíšek <mvojtisek@dnanexus.com>
Co-authored-by: Jiří Lagan <43816355+jirkalagan@users.noreply.github.com>
Co-authored-by: Pavel Kubal <pkubal@dnanexus.com>
Co-authored-by: Jiří Lagan <43816355+jirkalagan@users.noreply.github.com>
Co-authored-by: Jiří Lagan <43816355+jirkalagan@users.noreply.github.com>
Co-authored-by: Pavel Kubal <pkubal@dnanexus.com>
Co-authored-by: Shifaiz Mohamed <smohamed@dnanexus.com>
…ect (#3991)

Co-authored-by: Jiří Lagan <43816355+jirkalagan@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jiří Lagan <43816355+jirkalagan@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Jiří Lagan <43816355+jirkalagan@users.noreply.github.com>
Co-authored-by: rrayavara <rrayavarapu@dnanexus.com>
@gitbisect
gitbisect requested review from Copilot and rrayavara June 10, 2026 16:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.


flash.discard if messages.any?

render json: { meta: { messages: messages } }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Style/HashSyntax: Omit the hash value.

@@ -18,8 +18,7 @@ class ApiController < ApplicationController
before_action :require_api_login, except: %i(destroy)
before_action :validate_create_asset, only: :create_asset
before_action :check_total_and_job_charges_limit, only: %i(run_workflow)
before_action :check_total_charges_limit, only: %i(create_file create_asset)
before_action :validate_create_file, only: :create_file
before_action :check_total_charges_limit, only: %i(create_asset)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Metrics/ClassLength: Class has too many lines. [852/300]

@gitbisect
gitbisect marked this pull request as draft June 10, 2026 16:35
@gitbisect gitbisect closed this Jun 10, 2026
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.

8 participants