Skip to content

Failure to commit through web-ui after deleting added file #481

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

Closed
isc-pbarton opened this issue Sep 17, 2024 · 2 comments · Fixed by #579
Closed

Failure to commit through web-ui after deleting added file #481

isc-pbarton opened this issue Sep 17, 2024 · 2 comments · Fixed by #579
Assignees
Labels
bug Something isn't working customer Based on customer feedback (as opposed to something identified by developers) usability

Comments

@isc-pbarton
Copy link
Collaborator

isc-pbarton commented Sep 17, 2024

Steps to reproduce:

  1. Run a baseline export without committing: do ##class(SourceControl.Git.API).BaselineExport("")
  2. Confirm in the Git WebUI that some files were added to source control
  3. From the file system, delete one of the added files.
  4. In the Git WebUI, check "select all files" and attempt to commit.

I get an error message:

error: pathspec 'cls/pbarton7/test50.cls' did not match any file(s) known to git

The use case this might be an issue for is if I have a large baseline export, but there are some files I don't want included so I delete them from the file system. This puts us into an edge case where committing should work but does not.

@isc-tleavitt isc-tleavitt added the bug Something isn't working label Sep 17, 2024
@isc-tleavitt
Copy link
Collaborator

I don't see this as high priority because you need to delete from the filesystem. Re-running git status (which happens in a bunch of different ways) should clear things up in the uncommitted queue. If it doesn't then I'd bump the priority on this to Q3/Q4.

@isc-tleavitt isc-tleavitt added customer Based on customer feedback (as opposed to something identified by developers) usability labels Oct 11, 2024
@isc-tleavitt isc-tleavitt added this to the Q3/Q4 2024 Targets milestone Oct 11, 2024
@isc-tleavitt
Copy link
Collaborator

@isc-pbarton we saw this on our customer call Wednesday afternoon, so upped priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working customer Based on customer feedback (as opposed to something identified by developers) usability
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants