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

Align wot-related topics across all repositories #13

Merged
merged 2 commits into from
Dec 1, 2023

Conversation

JKRhb
Copy link
Member

@JKRhb JKRhb commented Dec 1, 2023

As discussed in today's committer meeting, this PR adds the topics wot and web-of-things to all of our repos that did not have them previously.

I wasn't sure if we said that the webofthings topic that is currently set for dart_wot should be replaced/removed – if so, then just let me know and I will update the PR accordingly :)

@JKRhb JKRhb requested review from a team as code owners December 1, 2023 12:06
@JKRhb
Copy link
Member Author

JKRhb commented Dec 1, 2023

This comment has been minimized.

otterdog/eclipse-thingweb.jsonnet Outdated Show resolved Hide resolved
Copy link
Member

@danielpeintner danielpeintner left a comment

Choose a reason for hiding this comment

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

LGTM

No strong opinion on the proposed removal from @egekorkan

Co-authored-by: Ege Korkan <[email protected]>
Signed-off-by: Jan Romann <[email protected]>
Copy link

github-actions bot commented Dec 1, 2023

Diff for 6b1c343:
Printing local diff for configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json'

Actions are indicated with the following symbols:
+   create
!   modify
!   forced update
-   delete

Organization eclipse-thingweb[id=eclipse-thingweb]
  there have been 5 validation infos, enable verbose output with '-v' to to display them.

  
!   repository[name=".github"] {
!     topics                                                   = "['iot', 'organization', 'web', 'web-of-things']" -> "['iot', 'organization', 'web', 'web-of-things', 'wot']"
!   }

  
!   repository[name="dart_wot"] {
!     topics                                                   = "['coap', 'dart', 'flutter', 'internetofthings', 'iot', 'webofthings', 'wot']" -> "['coap', 'dart', 'flutter', 'internetofthings', 'iot', 'web-of-things', 'wot']"
!   }

  
!   repository[name="node-wot"] {
!     topics                                                   = "['iot', 'nodejs', 'web', 'web-of-things']" -> "['iot', 'nodejs', 'web', 'web-of-things', 'wot']"
!   }

  
!   repository[name="playground"] {
!     topics                                                   = "['iot', 'nodejs', 'web', 'web-of-things']" -> "['iot', 'nodejs', 'web', 'web-of-things', 'wot']"
!   }

  
!   repository[name="test-things"] {
!     topics                                                   = "['iot', 'protocols', 'testing', 'web-of-things']" -> "['iot', 'protocols', 'testing', 'web-of-things', 'wot']"
!   }

  
!   repository[name="thingweb"] {
!     topics                                                   = "['iot', 'organization', 'web', 'web-of-things']" -> "['iot', 'organization', 'web', 'web-of-things', 'wot']"
!   }

  
!   repository[name="website"] {
!     topics                                                   = "['iot', 'web', 'web-of-things']" -> "['iot', 'web', 'web-of-things', 'wot']"
!   }
  
  Plan: 0 to add, 7 to change, 0 to delete.
Showing diff to a canonical version of the configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json'

Organization eclipse-thingweb[id=eclipse-thingweb]

@netomi netomi merged commit 869232a into eclipse-thingweb:main Dec 1, 2023
2 checks passed
@netomi
Copy link
Contributor

netomi commented Dec 1, 2023

changes are live.

@JKRhb JKRhb deleted the update-topics branch December 1, 2023 19:09
@JKRhb
Copy link
Member Author

JKRhb commented Dec 1, 2023

Thank you, @netomi!

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