Skip to content

Add Firebase - #997

Merged
jbampton merged 1 commit into
brisbanesocialchess:mainfrom
jbampton:temp
Feb 28, 2026
Merged

jbampton merged 1 commit into
brisbanesocialchess:mainfrom
jbampton:temp

Conversation

@jbampton

Copy link
Copy Markdown
Contributor

Pull Request

PR Checklist

  • I have tested my changes locally.
  • I have run pre-commit run --all-files locally.
  • I have updated the documentation if applicable.
  • I have reviewed my code for any potential issues.

PR Type

  • Bugfix
  • Feature
  • Style update
  • Refactor
  • Docs
  • Other (please describe)

Current behavior

Fixes: #
Part of: #

New behavior

Screenshots (if applicable)

Additional context

@deepsource-io

deepsource-io Bot commented Feb 28, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 1e5afe0...c58c8c7 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Docker Feb 28, 2026 8:46p.m. Review ↗
JavaScript Feb 28, 2026 8:46p.m. Review ↗
Python Feb 28, 2026 8:46p.m. Review ↗
Secrets Feb 28, 2026 8:46p.m. Review ↗

@jbampton jbampton added this to Chess Feb 28, 2026
@jbampton jbampton moved this to In Progress in Chess Feb 28, 2026
@jbampton jbampton added this to the 2026 First Quarter milestone Feb 28, 2026
Comment thread .github/workflows/firebase-hosting-merge.yml Fixed
@jbampton
jbampton merged commit 7a3a48e into brisbanesocialchess:main Feb 28, 2026
34 of 37 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Chess Feb 28, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Chess Feb 28, 2026
@jbampton
jbampton deleted the temp branch February 28, 2026 20:51
Subham-KRLX pushed a commit to Subham-KRLX/brisbanesocialchess.github.io that referenced this pull request Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants