Skip to content

Commit

Permalink
eslint: remove linting warnings/errors
Browse files Browse the repository at this point in the history
  • Loading branch information
prestist committed Oct 27, 2023
1 parent 470826f commit 90c2d8c
Show file tree
Hide file tree
Showing 8 changed files with 28 additions and 20 deletions.
1 change: 1 addition & 0 deletions .github/workflows/linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,3 +42,4 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
TYPESCRIPT_DEFAULT_STYLE: prettier
VALIDATE_JSCPD: false
VALIDATE_MARKDOWN: false
14 changes: 9 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,30 +1,34 @@
# FCOS-Meeting-Action

Welcome to the Fedora CoreOS Meeting tracker. This repo serves as a tracker for creating issues to track upcoming FCOS meetings. The issues are generated every Wednesday, and are meant to be used by the community meeting host as a guide for running the meeting.

The meeting notes are formed from three major parts. The first part is the [meeting template](./static/meeting-template.md), which is the base of the meeting notes. The second, and third parts are formed from the topics in the [fedora-coreos-tracker](https://github.com/coreos/fedora-coreos-tracker) repo and the past action items from the prior community meeting [fedora-meetings](https://meetbot-raw.fedoraproject.org/teams/fedora_coreos_meeting/fedora_coreos_meeting).

The Fedora CoreOS Working Group works to bring together the various technologies and produce Fedora CoreOS.
Get Fedora CoreOS

The Fedora CoreOS Working Group has a weekly meeting. The meeting usually happens in #fedora-meeting-1 on irc.libera.chat (Webchat) and the schedule for the meeting can be found here: https://calendar.fedoraproject.org/CoreOS/ Currently, meetings are at 16:30 UTC on Wednesdays.
The Fedora CoreOS Working Group has a weekly meeting. The meeting usually happens in #fedora-meeting-1 on irc.libera.chat (Webchat) and the schedule for the meeting can be found in this [calendar](https://calendar.fedoraproject.org/CoreOS/) Currently, meetings are at 16:30 UTC on Wednesdays.

As the Matrix/IRC bridge is down, it is currently not possible to attend the meeting from a Matrix account and you have to join using IRC. You can use the Webchat to temporarily join the meeting on IRC.
Steps to run the meeting

Navigate to this week's meeting and follow the steps presented.

Working days: non-holiday weekdays. Relevant holidays are the national holidays of the USA, Western Europe, and India.
Navigate to this week's meeting and follow the steps presented.

Working days: non-holiday weekdays. Relevant holidays are the national holidays of the USA, Western Europe, and India.

# Development

## Prerequisites

- install node package manger (npm)
- use npm to install typescript `npm install typescript --save-dev`

## Dev Workflow

- Work against the `.ts` files to add your changes
- Compile them using `npm run bundle` which produces `.js` files
- Commit the `.ts` and `.js` files

### Note
The action uses the `.js` files to run the action, and are found in the `dist directory. The `.ts` files are used for development purposes only.

The action uses the `.js` files to run the action, and are found in the `dist` directory. The `.ts` files are used for development purposes only.
14 changes: 7 additions & 7 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
"format:write": "prettier --write **/*.ts",
"format:check": "prettier --check **/*.ts",
"lint": "npx eslint . -c ./.github/linters/.eslintrc.yml",
"lint:fix": "npx eslint . -c ./.github/linters/.eslintrc.yml --fix",
"package": "ncc build src/index.ts --license licenses.txt",
"package:watch": "npm run package -- --watch",
"test": "(jest && make-coverage-badge --output-path ./badges/coverage.svg) || make-coverage-badge --output-path ./badges/coverage.svg",
Expand Down
6 changes: 3 additions & 3 deletions src/actionItems.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,18 +14,18 @@ export async function GetActionItems(): Promise<string> {
const meetingNotesURL = core.getInput('rootURLMeetingLogs')
let lastMeetingNotesUrl = `fedora_coreos_meeting.`
const listOfMeetings = await fetchData(meetingNotesURL)
let matches = listOfMeetings.match(meetingListRegEx)
const matches = listOfMeetings.match(meetingListRegEx)

if (matches != null) {
const lastMeeting = matches[matches.length - 1]
// This should be the latest meeting`s date in with the format of YYYY-MM-DD-HH.MM.txt
lastMeetingNotesUrl = meetingNotesURL + lastMeetingNotesUrl + lastMeeting
console.debug('last meeting notes url' + lastMeetingNotesUrl)
console.debug(`last meeting notes url${lastMeetingNotesUrl}`)
const lastMeetingNotes = await fetchData(lastMeetingNotesUrl)
const actionItemMatches = actionItemsRegEx.exec(lastMeetingNotes)

if (actionItemMatches) {
console.debug('action item matches' + actionItemMatches[0])
console.debug(`action item matches${actionItemMatches[0]}`)
// if the match is just new lines, then there were no action items
if (actionItemMatches[0].match(/^\s*$/)) {
return `#topic there are no action items from the last meeting.`
Expand Down
2 changes: 1 addition & 1 deletion src/createIssue.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ export async function createThisReposIssue(body: string): Promise<void> {
})
// calculate todays date in YYYY-MM-DD format
const today = new Date().toISOString().split('T')[0]
var title = core.getInput('issueTitle') + ' ' + today
const title = `${core.getInput('issueTitle')} ${today}`
const githubRepository = process.env.GITHUB_REPOSITORY
if (!githubRepository) {
throw new Error(`GITHUB_REPOSITORY environment variable is not set`)
Expand Down
6 changes: 3 additions & 3 deletions src/meetingTopics.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@ export async function GetMeetingTopics(): Promise<string> {
}

let issuesToBeDiscussed = ``
for (let i = 0; i < issues.data.length; i++) {
issuesToBeDiscussed += ` - [ ] \`#topic ${issues.data[i].title}\` \n`
issuesToBeDiscussed += ` - \`#link ${issues.data[i].html_url}\` \n`
for (const i of issues.data) {
issuesToBeDiscussed += ` - [ ] \`#topic ${i.title}\` \n`
issuesToBeDiscussed += ` - \`#link ${i.html_url}\` \n`
}
return issuesToBeDiscussed
} catch (error) {
Expand Down
4 changes: 3 additions & 1 deletion static/meeting-template.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
# FCOS-Meeting

## Meeting Notes
The meeting is held every week, meetings are at `16:30 UTC` on Wednesdays. The meeting is held in #fedora-meeting-1 on libera.chat. The meeting is logged and the logs are available at https://meetbot.fedoraproject.org/teams/fedora_coreos_meeting/

The meeting is held every week, meetings are at `16:30 UTC` on Wednesdays. The meeting is held in #fedora-meeting-1 on libera.chat. The meeting is logged and the logs are available [here](https://meetbot.fedoraproject.org/teams/fedora_coreos_meeting/)

### Required Concepts

Expand Down Expand Up @@ -50,6 +51,7 @@ At least 5 people must vote, or 51% of the WG membership, whichever is less. Vot


## Meeting Steps

1. Join channel #fedora-meeting-1 on libera.chat copy these commands and paste them in the channel

- [ ] `#startmeeting fedora_coreos_meeting`
Expand Down

0 comments on commit 90c2d8c

Please sign in to comment.