Skip to content

Update local dev aliases and ignores - #65

Merged
mandaiy merged 1 commit into
mainfrom
staged-dotfiles-updates
May 6, 2026
Merged

Update local dev aliases and ignores#65
mandaiy merged 1 commit into
mainfrom
staged-dotfiles-updates

Conversation

@mandaiy

@mandaiy mandaiy commented May 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • set AWS_REGION for the awslocal alias
  • add a git-main-worktree helper alias
  • ignore local Claude settings and .envrc files

Tests

  • pre-commit hooks ran during commit

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9fbeb080b6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread fish/config.fish

alias nvimdiff="nvim -d"
alias awslocal="AWS_ACCESS_KEY_ID=dummy AWS_SECRET_ACCESS_KEY=dummy aws --endpoint-url=http://localhost:4566"
alias awslocal="AWS_REGION=ap-northeast-1 AWS_ACCESS_KEY_ID=dummy AWS_SECRET_ACCESS_KEY=dummy aws --endpoint-url=http://localhost:4566"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep awslocal default region compatible with common LocalStack flows

Hardcoding AWS_REGION=ap-northeast-1 in the awslocal alias can break common LocalStack command sequences that assume us-east-1 defaults (for example, S3 bucket creation commands that omit a location constraint). This is a behavioral regression from the prior alias because every invocation is now forced into a non-default region, so existing scripts and one-liners that previously worked can start failing unless callers add explicit region-specific flags.

Useful? React with 👍 / 👎.

@mandaiy
mandaiy merged commit 1dacad4 into main May 6, 2026
1 check passed
@mandaiy
mandaiy deleted the staged-dotfiles-updates branch May 6, 2026 06:15
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