Skip to content

Fix cargo install failure and add Copilot instructions#474

Open
orgcontrib wants to merge 2 commits into
jansc:masterfrom
orgcontrib:copilot/add-copilot-instructions
Open

Fix cargo install failure and add Copilot instructions#474
orgcontrib wants to merge 2 commits into
jansc:masterfrom
orgcontrib:copilot/add-copilot-instructions

Conversation

@orgcontrib

Copy link
Copy Markdown

Summary

Fixes issue #473: cargo install ncgopher was failing due to a breaking change in the time crate.

Changes

  1. Fix cargo install failure: Replace removed OffsetDateTime::now_local() with OffsetDateTime::now_utc() in src/main.rs and src/controller.rs
  2. Add Copilot instructions: New .github/copilot-instructions.md documenting build/test commands, architecture, and repository conventions
  3. Fix bookmark menu bug: Corrected use of variable in remove_bookmark_action (use entry.title instead of b.title)

Verification

  • cargo install --path . now succeeds
  • Binary runs: ncgopher 0.8.0
  • No breaking changes to existing functionality

Related

Fixes #473

orgcontrib and others added 2 commits July 14, 2026 00:21
- Add .github/copilot-instructions.md to document build/test/architecture/conventions for Copilot sessions
- Fix incorrect variable use in remove_bookmark_action (use entry.title)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The time crate v0.3.53 removed OffsetDateTime::now_local(). Replace with
OffsetDateTime::now_utc() in src/main.rs and src/controller.rs.

Fixes: cargo install ncgopher (issue jansc#473)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

no associated function or constant named now_local found for struct OffsetDateTime in the current scope

1 participant