Skip to content

Conversation

@sivaraam
Copy link
Member

@sivaraamsivaraam commented Mar 1, 2024

Motivated by comments of Markus and Christian in the discussion of edition 108 ref 1.

@chriscool
Copy link
Collaborator

chriscool commented Mar 1, 2024

Motivated by comments of Martin and Chrsitian in the discussion of edition 108 ref 1.

@sivaraam I think you mean "Markus" and "Christian". It would be nice if you could fix that and squash the 3 commits together. Otherwise I am fine with merging this PR. Thanks!

@sivaraamsivaraamforce-pushed the support-article-include-dijsktra-reference branch 2 times, most recently from 426a4d0 to 8dde840CompareMarch 1, 2024 14:11
@sivaraam
Copy link
MemberAuthor

sivaraam commented Mar 1, 2024

Sorry about that typo and redundant commits (was editing on mobile and was able to only add new commits for the fixes 😅 )

Fixed and squashed the commits. 👍

Motivated by comments of Markus and Christian in the discussion of edition 108 [ref 1]. [ref 1]: #686 (comment)
@sivaraamsivaraamforce-pushed the support-article-include-dijsktra-reference branch from 8dde840 to 73796bcCompareMarch 1, 2024 14:15
@chriscoolchriscool merged commit 0152193 into masterMar 1, 2024
@chriscool
Copy link
Collaborator

@sivaraam thanks for fixing everything! Merged!

Sign up for freeto 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

@sivaraam@chriscool