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.

Sign up for GitHub

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

Jump to bottom

DOPS-2746 Add sonar, dojo #796

Closed
wants to merge 37 commits into from
Closed

DOPS-2746 Add sonar, dojo #796

wants to merge 37 commits into from

Conversation

C4tWithShell
Copy link
Contributor

  1. Set up sonar
  2. Set up dojo
  3. Fixed problem with coverage, change to lcov format

Merge after defectdojo update!

C4tWithShell and others added 14 commits September 18, 2023 16:32
@C4tWithShell C4tWithShell self-assigned this Oct 23, 2023
@Alexey-N-Chernyshov
Copy link
Contributor

Alexey-N-Chernyshov commented Oct 23, 2023 •
edited
Loading

Contracts were moved to https://github.com/sora-xor/sora2-federated-bridge-evm-contracts.git, see #780.
There is no need to run Slither in this repo.

wpi86
wpi86 requested changes Oct 24, 2023
View reviewed changes
Jenkinsfile Outdated Show resolved Hide resolved
Alexey-N-Chernyshov
Alexey-N-Chernyshov requested changes Oct 24, 2023
View reviewed changes
Copy link
Contributor

@Alexey-N-Chernyshov Alexey-N-Chernyshov left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Solidity was moved to https://github.com/sora-xor/sora2-federated-bridge-evm-contracts in #685

@C4tWithShell C4tWithShell changed the base branch from master to develop February 7, 2024 10:19
@C4tWithShell
Copy link
Contributor Author

C4tWithShell commented Feb 13, 2024 •
edited
Loading

Created PR for wrong branch. Created a new PR - #903

@C4tWithShell C4tWithShell deleted the feature/DOPS-2746/sonar_dojo branch February 13, 2024 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@f33r0 f33r0 f33r0 approved these changes

@alexkg alexkg alexkg approved these changes

@wpi86 wpi86 wpi86 approved these changes

@BAStos525 BAStos525 BAStos525 approved these changes

@Cre-eD Cre-eD Cre-eD approved these changes

@Alexey-N-Chernyshov Alexey-N-Chernyshov Alexey-N-Chernyshov approved these changes

@baotn166 baotn166 Awaiting requested review from baotn166

@vovac12 vovac12 Awaiting requested review from vovac12

@pierrepicaud pierrepicaud Awaiting requested review from pierrepicaud

@safinsaf safinsaf Awaiting requested review from safinsaf

@Naghme98 Naghme98 Awaiting requested review from Naghme98

@marat-biriushev marat-biriushev Awaiting requested review from marat-biriushev

@RamilMus RamilMus Awaiting requested review from RamilMus

@stefashkaa stefashkaa Awaiting requested review from stefashkaa

@Nikita-Polyakov Nikita-Polyakov Awaiting requested review from Nikita-Polyakov

Assignees

@C4tWithShell C4tWithShell

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@C4tWithShell @Alexey-N-Chernyshov @Cre-eD @BAStos525 @wpi86 @alexkg @f33r0 @vovac12 @wer1st @marat-biriushev