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.

Sign up for GitHub

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

Jump to bottom

update controller doc examples #301

Merged
merged 7 commits into from
Sep 20, 2024
Merged

update controller doc examples #301

merged 7 commits into from
Sep 20, 2024

Conversation

decentralgabe
Copy link
Collaborator

@decentralgabe decentralgabe commented Sep 17, 2024 •
edited by pr-preview bot
Loading

fix #154

Preview | Diff

TallTed
TallTed requested changes Sep 17, 2024
View reviewed changes
Copy link
Member

@TallTed TallTed left a comment

Choose a reason for hiding this comment

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

Editorial, for consistency & clarity

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
selfissued and others added 5 commits September 17, 2024 15:54
@selfissued @TallTed
Applied Ted's suggestion
372808d 
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@selfissued @TallTed
Applied Ted's suggestion
5f7bb04 
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@selfissued @TallTed
Applied Ted's suggestion
bb4f450 
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@selfissued @TallTed
Applied Ted's suggestion
fcd879e 
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@selfissued @TallTed
Applied Ted's suggestion
71ea593 
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Merge remote-tracking branch 'origin/main' into issue-154
65e5d84 
* origin/main:
  Move Conformance section into Introduction (#299)
  Remove my name from this spec (#295)
  Add Acknowledgements section (#298)
  update respec (#303)

# Conflicts:
#	index.html
@decentralgabe
Copy link
Collaborator Author

Updating examples, nothing normative. Editorial approval. Changes and merge conflicts addressed. Merging.

@decentralgabe decentralgabe merged commit e0e7d91 into main Sep 20, 2024
2 checks passed
@decentralgabe decentralgabe deleted the issue-154 branch September 20, 2024 20:51
decentralgabe pushed a commit that referenced this pull request Sep 20, 2024
Merge remote-tracking branch 'origin/main' into fix-300
a9a7ee9 
* origin/main:
  update controller doc examples (#301)
  Move Conformance section into Introduction (#299)
  Remove my name from this spec (#295)
  Add Acknowledgements section (#298)
  update respec (#303)

# Conflicts:
#	index.html
decentralgabe pushed a commit that referenced this pull request Sep 20, 2024
Merge remote-tracking branch 'origin/main' into issue-296
d28ab37 
* origin/main:
  update controller doc examples (#301)
  Move Conformance section into Introduction (#299)
  Remove my name from this spec (#295)
  Add Acknowledgements section (#298)

# Conflicts:
#	index.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TallTed TallTed TallTed requested changes

@selfissued selfissued selfissued approved these changes

@brentzundel brentzundel Awaiting requested review from brentzundel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Controller document examples
3 participants
@decentralgabe @TallTed @selfissued