Skip to content

feat(sec): move to LFX#223

Open
avivkeller wants to merge 1 commit intomainfrom
sec/lfx
Open

feat(sec): move to LFX#223
avivkeller wants to merge 1 commit intomainfrom
sec/lfx

Conversation

@avivkeller
Copy link
Member

Updated meeting details for the Node.js Security Working Group, including new calendar filter and ICAL URL.
Copilot AI review requested due to automatic review settings March 22, 2026 20:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Node.js Security WG meeting template to use the Linux Foundation LFX calendar feed and Zoom LFX joining link (per nodejs/security-wg#1558), replacing the previous Google Calendar iCal source and placeholder joining instructions.

Changes:

  • Switched ICAL_URL from Google Calendar to an LFX webcal endpoint.
  • Updated CALENDAR_FILTER to match the new LFX event naming.
  • Replaced the participant join link with the Zoom LFX meeting URL.
Comments suppressed due to low confidence (1)

templates/meeting_base_security-wg:11

  • JOINING_INSTRUCTIONS starts a quoted multi-line value but is never closed with a terminating ". dotenv.parse() requires the closing quote for multi-line values; as-is this will break meeting config parsing for security-wg (and may cause subsequent properties to be misparsed). Add the closing quote line (consistent with other meeting_base_* templates) so the template remains valid dotenv syntax.
JOINING_INSTRUCTIONS="

* link for participants: <https://zoom-lfx.platform.linuxfoundation.org/meeting/92626800362?password=78ffb026-dfb9-4549-b8cd-c3b82bd907f7>
* For those who just want to watch We stream our conference call straight to YouTube so anyone can listen to it live, it should start playing at <https://www.youtube.com/c/nodejs+foundation/live> when we turn it on. There's usually a short cat-herding time at the start of the meeting and then occasionally we have some quick private business to attend to before we can start recording & streaming. So be patient and it should show up.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants