-
Notifications
You must be signed in to change notification settings - Fork 55
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
fix: community call #1516
fix: community call #1516
Conversation
Signed-off-by: Alejandro Jose Leiva Palomo <alejandro.leiva.palomo@ibm.com>
Signed-off-by: Alejandro Jose Leiva Palomo <alejandro.leiva.palomo@ibm.com>
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
Reviewers - please suggest improvements, if any - thx! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a couple of suggestions, but nothing that would block this change. Just discussion points.
|
||
- Meeting agenda and notes: [here](https://docs.google.com/document/d/1z9xvt-Z97j4CtEH1-nR9sMWul7jQkUi_fNY7BdMPgxM/edit#heading=h.nohkp1kbeduj) | ||
|
||
##### Chat anytime |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not block this particular change, but since the Slack and Zoom meeting are for the oscal-compass
organization, I wonder if this type of information would belong in an organization profile or in a community repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where would this go, when I visit https://github.com/oscal-compass? Also, replicating information seemed doomed to inconsistency, so perhaps one should point to the other.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can have an organizational profile READMEs so the information would be available when the organization link is visited. Example here for the CNCF org - https://github.com/cncf/.github/tree/master/profile. Totally agree that we should not duplicate the information, but link to it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So I think you mean here https://github.com/oscal-compass/.github in outrcase. Seems fine to me, but then perhaps we should open an issue against that repo and make a proposal as to what it should contain, pre-CNCF.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good. I can open that issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added here oscal-compass/.github#1
README.md
Outdated
|
||
- Meeting Id: 927 2923 5315 | ||
|
||
- Meeting agenda and notes: [here](https://docs.google.com/document/d/1z9xvt-Z97j4CtEH1-nR9sMWul7jQkUi_fNY7BdMPgxM/edit#heading=h.nohkp1kbeduj) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps we should add a process on how to get topics on the agenda
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we have an official process yet...but once we do we can certainly add words to the README.md community meetings section, if that make sense.
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
…/compliance-trestle.git into fix/community-call
Dear Reviewers: please see changes since initial delivery.
|
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
Signed-off-by: Alejandro Jose Leiva Palomo <alejandro.leiva.palomo@ibm.com>
Signed-off-by: Alejandro Jose Leiva Palomo <alejandro.leiva.palomo@ibm.com>
@degenaro Ash from CNCF has changed the zoom settings so we are able to use a Passcode instead of doing a login. I have added the Passcode in the latest commit as well as changed the wording for Zoom login, please take a quick look before merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Alejandro Jose Leiva Palomo <alejandro.leiva.palomo@ibm.com>
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
Types of changes
develop
->main
)Quality assurance (all should be covered).
Summary
Key links:
Before you merge