Skip to content
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.

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

fix: fetch latest changes and update key mappings #112

Merged
merged 3 commits into from
Aug 26, 2024
Merged

Conversation

dgokcin
Copy link
Owner

@dgokcin dgokcin commented Aug 26, 2024

Summary

This pull request includes two main changes: fetching the latest changes from the remote repository before generating a diff and updating key mappings in visual mode.

Changes

  • Added a command to fetch the latest changes from the remote repository in copilot-chat.lua.
  • Updated the example output in the documentation to properly escape backticks.
  • Split the key mappings for disabling convert to uppercase and lowercase in visual mode in vscode.lua.

Additional Notes

  • The fetch command ensures that the diff is generated against the latest changes from the remote repository.
  • The key mappings update in vscode.lua improves clarity and maintainability.

… in visual mode

previously, both 'u' and 'U' were disabled together in visual mode. this change separates them into individual keymap settings to ensure they are disabled independently.
corrected the example for creating a pull request using the github cli in the copilot-chat plugin documentation. changes include:
- replaced single quotes with double quotes in the example command
- added escaped backticks in the body of the pull request example
- added a command to fetch the latest changes from the remote repository
- ensures that the diff is generated against the most recent state of the main branch
@dgokcin dgokcin merged commit 48dc110 into main Aug 26, 2024
1 of 2 checks passed
@dgokcin dgokcin deleted the tiny-error-fix branch August 26, 2024 21:14
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