Skip to content

Conversation

filipecosta90
Copy link
Collaborator

No description provided.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 12, 2022

This pull request introduces 1 alert when merging 313b455 into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 29, 2022

This pull request introduces 1 alert when merging b87c228 into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 31, 2022

This pull request introduces 1 alert when merging 24d876b into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 31, 2022

This pull request introduces 1 alert when merging fb12069 into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 31, 2022

This pull request introduces 1 alert when merging 67d7fb5 into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 31, 2022

This pull request introduces 1 alert when merging 80973ef into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Nov 14, 2022

This pull request introduces 1 alert when merging 5372628 into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Nov 14, 2022

This pull request introduces 1 alert when merging 9a70f5c into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Nov 14, 2022

This pull request introduces 1 alert and fixes 7 when merging 5b9fcc9 into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

fixed alerts:

  • 6 for Unused local variable
  • 1 for Unused import

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Nov 14, 2022

This pull request introduces 1 alert and fixes 7 when merging 01de53d into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

fixed alerts:

  • 6 for Unused local variable
  • 1 for Unused import

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Nov 14, 2022

This pull request introduces 1 alert and fixes 7 when merging 3aa26a4 into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

fixed alerts:

  • 6 for Unused local variable
  • 1 for Unused import

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Nov 14, 2022

This pull request introduces 1 alert and fixes 7 when merging bcb939a into 5ac28bb - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

fixed alerts:

  • 6 for Unused local variable
  • 1 for Unused import

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@filipecosta90 filipecosta90 changed the title [In-Progress] CI fixes CI fixes Nov 14, 2022
@filipecosta90 filipecosta90 merged commit bdd34fa into master Nov 14, 2022
@filipecosta90 filipecosta90 deleted the ci.fixes branch November 14, 2022 20:46
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