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 Types Team calendar #68

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Update Types Team calendar #68

merged 1 commit into from
Nov 13, 2024

Conversation

apiraino
Copy link
Contributor

@apiraino apiraino commented Nov 13, 2024

Removed the weekly occurrence from two Types team meetings that are not taking place anymore, this should be enough to remove them from our calenders.

(agreed this change with @lcnr )

r? @tgross35

thanks!

Copy link
Contributor

@lcnr lcnr left a comment

Choose a reason for hiding this comment

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

or wait, the second one may still happen cc @compiler-errors

only the first one has definitely been removed. Is there a reason to not entirely delete these entries?

@apiraino
Copy link
Contributor Author

I thought leaving them inactive in case you folks decide to start them again. But it's up to you, we can also remove them.

@lcnr
Copy link
Contributor

lcnr commented Nov 13, 2024

i'd remove the first one. It's highly unlikely we'll reuse the same meeting in the future

types.toml Show resolved Hide resolved
@apiraino apiraino force-pushed the update-types-kalender branch from ca4f447 to ee32163 Compare November 13, 2024 19:44
@tgross35
Copy link

r? @tgross35

I'm not on the types team, was this the right r? :)

@rustbot

This comment was marked as resolved.

@apiraino
Copy link
Contributor Author

oh no, pinged the wrong person. Again 😆 Sorry Trevor.

@apiraino apiraino merged commit 4afc4d5 into main Nov 13, 2024
1 check passed
@tgross35 tgross35 deleted the update-types-kalender branch November 13, 2024 20: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.

5 participants