Skip to content

Conversation

bertux
Copy link
Contributor

@bertux bertux commented Apr 4, 2025

Generated automatically by GitHub Security recommendations.

Generated automatically by GitHub Security recommendations.

Signed-off-by: Bertrand Juglas <b@juglas.name>
@bertux bertux requested a review from Copilot April 4, 2025 21:31
Copy link

vercel bot commented Apr 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
allo-yeeter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 4, 2025 9:32pm

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

.github/workflows/codeql.yml:98

  • The 'category' parameter includes a leading slash which may result in unintended behavior. Please verify if this slash was intended and remove it if not.
category: "/language:${{matrix.language}}"

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@bertux
Copy link
Contributor Author

bertux commented Apr 5, 2025

To make it cleaner about what we can change we should use git submodules for the packages we use like Allo Kit and the libraries managed by Foundry in the contracts.

@Oba-One Oba-One self-requested a review April 9, 2025 19:25
@Oba-One Oba-One changed the base branch from main to develop April 9, 2025 19:30
@Oba-One Oba-One merged commit ffe85c4 into develop Apr 9, 2025
4 checks passed
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.

2 participants