Skip to content

Exclude dry activate#1

Open
mran-dbc wants to merge 6 commits into
masterfrom
exclude-dry-activate
Open

Exclude dry activate#1
mran-dbc wants to merge 6 commits into
masterfrom
exclude-dry-activate

Conversation

@mran-dbc
Copy link
Copy Markdown

No description provided.

@mran-dbc mran-dbc requested a review from adamtulinius May 21, 2026 08:19
@adamtulinius
Copy link
Copy Markdown

Kind reminder: Hostmap is meant to be used by others than DBC, so we should try to make changes that can be upstreamed. Therefore we need to remove all references to DBC, platform, and so on, also including names of actual servers. :)

Comment thread .gitignore
Comment on lines +9 to +10
/postgres.log
/.dev_postgres
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Maybe we can bundle all the pgsql lines together

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Like this?

/target
/.env
/.pre-commit-config.yaml
/result
/setup.sh

# DB stuff
/pg
/postgres.log
/.dev_postgres
/scratch.sql

# ide's
/.idea
/.vscode

# Temps
/api-key.txt
/test-assets/
todo.md

# direnv
.direnv/*

Comment thread README.md
nix develop
pg_initial_setup
cargo sqlx migrate run
cargo run -- test-assets/minimalTargetList.json
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I get this output:

❯ cargo run -- test-assets/minimalTargetList.json
+ command cargo run -- test-assets/minimalTargetList.json
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.12s
     Running `target/debug/hostmap test-assets/minimalTargetList.json`
2026-05-21T08:51:43.915078Z  INFO hostmap: Log level set to: hostmap=debug,info
error: unrecognized subcommand 'test-assets/minimalTargetList.json'

Usage: hostmap <COMMAND>

For more information, try '--help'.

@adamtulinius
Copy link
Copy Markdown

Kind reminder: Hostmap is meant to be used by others than DBC, so we should try to make changes that can be upstreamed. Therefore we need to remove all references to DBC, platform, and so on, also including names of actual servers. :)

I guess what I'm trying to say is to be mindful about how we can upstream this after merging it here

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