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

Update TSC companies for JS #4162

Merged
merged 12 commits into from
Sep 16, 2024
Merged

Update TSC companies for JS #4162

merged 12 commits into from
Sep 16, 2024

Conversation

datajoely
Copy link
Contributor

Checklist

  • Read the contributing guidelines
  • Signed off each commit with a Developer Certificate of Origin (DCO)
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added a description of this change in the RELEASE.md file
  • Added tests to cover my changes
  • Checked if this change will affect Kedro-Viz, and if so, communicated that with the Viz team

Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
datajoely and others added 2 commits September 13, 2024 16:41
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
Signed-off-by: Juan Luis Cano Rodríguez <juan_luis_cano@mckinsey.com>
Copy link
Member

@astrojuanlu astrojuanlu left a comment

Choose a reason for hiding this comment

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

❤️

Can we undo the ToC changes? (I think it's the Markdown extension for VS Code)

datajoely added 4 commits September 13, 2024 18:17
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
Copy link
Member

@astrojuanlu astrojuanlu left a comment

Choose a reason for hiding this comment

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

🙌🏼

Copy link
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

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

🌟

@datajoely datajoely enabled auto-merge (squash) September 16, 2024 13:14
Signed-off-by: Deepyaman Datta <deepyaman.datta@utexas.edu>
@deepyaman deepyaman enabled auto-merge (squash) September 16, 2024 14:26
datajoely added 3 commits September 16, 2024 15:30
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
Signed-off-by: datajoely <joel.schwarzmann@aneira.health>
@deepyaman deepyaman merged commit b60b02d into main Sep 16, 2024
41 checks passed
@deepyaman deepyaman deleted the docs/update-tsc branch September 16, 2024 15:17
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.

4 participants