You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The copilot-visualizer was developed outside of this repo, but there's interest in having all Copilot projects and libraries merged into a main repo for ease of management.
Type
Management: files need to be moved between repos.
Additional context
None.
Requester
Ivan Perez.
Method to check presence of bug
Not applicable (not a bug).
Expected result
The last state of the copilot-visualizer is included in this repository, disregarding history, and adjusting the changelog to make the old issues refer to copilot-visualizer#<ISSUE_NUM> so that they are not interpreted as related to the main Copilot repo.
Desired result
The last state of the copilot-visualizer is included in this repository, disregarding history, and adjusting the changelog to make the old issues refer to copilot-visualizer#<ISSUE_NUM> so that they are not interpreted as related to the main Copilot repo.
Proposed solution
Copy last state of copilot-visualizer as subdirectory of main repo.
Adjust Changelog in copilot-visualizer to make older issues point to correct repo.
The solution proposed produces the expected result. Details:
The following dockerfile checks out the latest version of copilot-visualizer, v4.4, and compares it against the version being merged, ignoring differences in the CHANGELOG (which is expected to be different) and any git-related files, and checking that there are no other differences, after which it prints the message "Success":
FROM ubuntu:noble
ENV DEBIAN_FRONTEND=noninteractive
RUN apt-get update
RUN apt-get install --yes git
RUN git config --global advice.detachedHead false
CMD git clone $REPO $NAME \
&& cd $NAME \
&& git checkout $COMMIT \
&& cd .. \
&& git clone -b v4.4 https://github.com/Copilot-Language/copilot-visualizer copilot-visualizer-original \
&& diff --exclude '.git' --exclude '.gitignore' --exclude 'CHANGELOG' -r $NAME/copilot-visualizer copilot-visualizer-original/ \
&& echo "Success"
Command (substitute variables based on new path after merge):
$ docker run -e REPO=https://github.com/Copilot-Language/copilot -e NAME=copilot -e COMMIT=b38f158b8a9ff437bd8458e3f6981ab59b69cf78 copilot-verify-624
Implementation is documented. Details:
No updates needed; change includes package in repo but purposefully leaves out any other updates.
Change history is clear.
Commit messages are clear.
Changelogs are updated.
CHANGELOG updated to make old issues point to original repo. No other changes needed to documentation.
Examples are updated. Details:
No updates needed.
Author is internal or has provided signed CLA.
Required version bumps are evaluated. Details:
Bump not needed; change includes existing package without affecting API.
Activity
ivanperez-keera commentedon Jun 27, 2025
Change Manager: Confirmed that the issue exists.
ivanperez-keera commentedon Jun 27, 2025
Technical Lead: Confirmed that the issue should be addressed.
ivanperez-keera commentedon Jun 27, 2025
Technical Lead: Issue scheduled for fixing in Copilot 4.5.
Fix assigned to: @ivanperez-keera .
copilot-visualizer: Include in mainline. Refs #624.
copilot-visualizer: Remove duplicate .gitignore. Refs #624.
copilot-visualizer: Adjust issue numbers in CHANGELOG. Refs #624.
copilot-visualizer: Document changes in CHANGELOG. Refs #624.
ivanperez-keera commentedon Jun 27, 2025
Implementor: Solution implemented, review requested.
ivanperez-keera commentedon Jun 28, 2025
Change Manager: Verified that:
Build log: https://app.travis-ci.com/github/Copilot-Language/copilot/builds/275607682
The following dockerfile checks out the latest version of copilot-visualizer, v4.4, and compares it against the version being merged, ignoring differences in the CHANGELOG (which is expected to be different) and any
git-related files, and checking that there are no other differences, after which it prints the message "Success":No updates needed; change includes package in repo but purposefully leaves out any other updates.
CHANGELOG updated to make old issues point to original repo. No other changes needed to documentation.
No updates needed.
Bump not needed; change includes existing package without affecting API.
ivanperez-keera commentedon Jun 28, 2025
Change Manager: Implementation ready to be merged.
Merge branch 'develop-visualizer'. Close Copilot-Language#624.