Skip to content

Refactor starboard into multiple crates #343

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
Draft
wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

Refactor starboard into multiple crates #343

wants to merge 22 commits into from

Conversation

circuitsacul
Copy link
Owner

@circuitsacul circuitsacul commented Aug 1, 2023 •
edited
Loading

Non-exhaustive todos:

  • Fix Dockerfile
  • docker-compose
  • Fix workflows
  • Add move_lock for starboard to pipelines
  • Testing of pipelines
  • Configure postgres instance (more connections? def more ram)

@circuitsacul
IT WORKS
a450ce2
@circuitsacul
Update Cargo.lock
e070040
@circuitsacul
Merge branch 'main' into compose
9995404
@circuitsacul
force macros to be used from crate root
4f09d96
@circuitsacul
workspace.resolver = 2
0df3be8
@circuitsacul
cargo fmt
263bd60
@circuitsacul
move move-locks for starboard to db pipeline
cd2a10b
@circuitsacul
fix database crate without backend feature
9adf47d
@circuitsacul
newline
9cef248
@circuitsacul
move sqlx-data.json
b5fa2e2
@circuitsacul
remove extra line
79384bf
@circuitsacul
update .env.example, create docker configs
fba9478
@circuitsacul
update & fix .env.example & docker-compose.yml
4ed04d0
@circuitsacul
double comment
05d47f3
@circuitsacul
move Dockerfile to crates/starboard
f4472c1
@circuitsacul
cargo fmt
b155fc9
@circuitsacul
move stuff around, add backup service
a86eb1f
@circuitsacul
use cargo chef for bot dockerfile
16ee180
@circuitsacul
fix validation errors, split parsing and validation (#348)
8c6939d 
* return enums instead of strings

* properly split parsing and validation
@circuitsacul
Merge branch 'main' into compose
9b48710
@circuitsacul
add tracing
bd83a6d
@circuitsacul
Merge branch 'main' into compose
e1ecd80
@circuitsacul circuitsacul marked this pull request as ready for review May 31, 2024 20:50
@circuitsacul circuitsacul marked this pull request as draft May 31, 2024 20:50
@circuitsacul circuitsacul reopened this May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
Starboard-4
Status: Triage
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@circuitsacul