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

Delete Old Branches #192

Open
jacobdbrown4 opened this issue Oct 19, 2022 · 5 comments
Open

Delete Old Branches #192

jacobdbrown4 opened this issue Oct 19, 2022 · 5 comments

Comments

@jacobdbrown4
Copy link
Collaborator

There are several branches that are "stale". We should go through these together and try to clean it up.

@jacobdbrown4
Copy link
Collaborator Author

jacobdbrown4 commented Apr 5, 2023

I have deleted several branches were old and had no new work on them. The leftover stale branches have some new things on them that we may still want. For example, the branch named "release1.5.0" has some documentation that we may want to move over before deleting.

We need to look at merging in the "case_insensitive_naming" branch. Same goes for the "pylint_jordi" branch.

@jacobdbrown4
Copy link
Collaborator Author

jacobdbrown4 commented Aug 30, 2023

The pylint_jordi branch has been moved over.

We still need to look at the "case_insensitive_naming" branch.

After we merge next_release into master, we can probably delete the following branches:

  • pylint_jordi
  • move_tests
  • move_tests_and_files

@jacobdbrown4 jacobdbrown4 added this to In Progress in Fall 2023 Release Aug 30, 2023
@jacobdbrown4
Copy link
Collaborator Author

jacobdbrown4 commented Sep 14, 2023

Ok here is an update on all the branches that will be merged/deleted eventually:

  • case_insensitive_naming - this branch contains code to tell the EDIF parser to be case sensitive for naming. I merged this code into next release but then removed it, so github thinks this branch has nothing new but it does.
  • test_unused_ports - this branch was made by @jgoeders . It looks like a test was added. There is also requirements.txt and a Makefile too.
  • release1.5.0 - this branch has some extra documentation that may be helpful to add to the main documentation.
  • converting_scripts - this branch appears to contain scripts to convert netlists between edif and verilog using Vivado
  • globalManager - contains some sort of manager
  • transformation-unit-test - contains old spydrnet stuff including some TMR stuff. I believe all of this is old and not needed.

@jacobdbrown4
Copy link
Collaborator Author

The script on branch 'converting_scripts' is also found in the master branch under support_files. See here. So this branch will be deleted.

@jacobdbrown4
Copy link
Collaborator Author

Ok here is an update on all the branches that will be merged/deleted eventually:

  • case_insensitive_naming - this branch contains code to tell the EDIF parser to be case sensitive for naming. I merged this code into next release but then removed it, so github thinks this branch has nothing new but it does.
  • test_unused_ports - this branch was made by @jgoeders . It looks like a test was added. There is also requirements.txt and a Makefile too.
  • globalManager - contains some sort of manager

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant