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

Initial revisions of project lifecycle #159

Merged
merged 3 commits into from
Apr 26, 2024

Conversation

bensternthal
Copy link
Contributor

This is an initial attempt to rework the project lifecycle process.

Note that once this is merged I will work on renaming the repo from TSC to TAC and adjusting all the links.

@bensternthal
Copy link
Contributor Author

Looks like the templates are hardcoded into check_projects_syntax.py, Ill have to work on that and file an updated pr.

Copy link

@cetola cetola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, and in alignment with what we discussed at the TAC.

@bensternthal
Copy link
Contributor Author

Yikes, OK the content and therefore process is tightly coupled to the code. This makes updates pretty hard, once I get this merged, perhaps we can chat about decoupling TAC process from the code that generates the website.

@bensternthal
Copy link
Contributor Author

I filed chipsalliance/chips-alliance-website#118 to address the path change I submitted in this PR.

I'd like @mgielda to review before I merge these.

@bensternthal
Copy link
Contributor Author

@mgielda I know you are super busy, but given the changes in this PR, it would be great to have your eyes on it before we merge.

@bensternthal bensternthal merged commit 09ed44f into main Apr 26, 2024
1 check passed
@bensternthal bensternthal deleted the project-application-updates branch April 26, 2024 20:19
@mgielda
Copy link
Member

mgielda commented May 1, 2024

Hi Ben, as mentioned in the latest TAC meeting, I was travelling to EW and then EOSS / CHIPS open HW event / LatchUp. If you want my review, you may have to ping me using another channel, as I have thousands of emails from GitHub. I was simply not aware of this PR. Overall, I believe the entirety of the TAC should have been made aware that the template is being changed by an email to the mailing list.

The code to build the website was specifically tied to the project templates to avoid discrepancies, so I would definitely oppose decoupling them again.

@bensternthal
Copy link
Contributor Author

@mgielda noted, I will send out notifications to the TAC mail list from now on for anything this substantial.

Understood about keeping things coupled.

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.

None yet

3 participants