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

[DO NOT MERGE] Test auto comment formatting #1268

Conversation

robojumper
Copy link
Member

Test if the change in #1266 had the desired effect.

@github-actions
Copy link
Contributor

Pull request modifies event listener templates

Difference (click to expand)
diff --git a/target/CHL_Event_Compiletest.uc b/target/CHL_Event_Compiletest.uc
index 119ff6b..c22d598 100644
--- a/target/CHL_Event_Compiletest.uc
+++ b/target/CHL_Event_Compiletest.uc
@@ -69,18 +69,18 @@ static function EventListenerReturn OnAllowRulerOnMission(Object EventData, Obje
 {
 	local XComGameState_MissionSite MissionState;
 	local XComLWTuple Tuple;
-	local bool bAllowRuler;
+	local bool bAllowRulerTempFormattingTestDoNotMerge;
 	local int RulerObjectID;
 
 	MissionState = XComGameState_MissionSite(EventSource);
 	Tuple = XComLWTuple(EventData);
 
-	bAllowRuler = Tuple.Data[0].b;
+	bAllowRulerTempFormattingTestDoNotMerge = Tuple.Data[0].b;
 	RulerObjectID = Tuple.Data[1].i;
 
 	// Your code here
 
-	Tuple.Data[0].b = bAllowRuler;
+	Tuple.Data[0].b = bAllowRulerTempFormattingTestDoNotMerge;
 
 	return ELR_NoInterrupt;
 }
What? (click to expand)

The Highlander documentation tool generates event listener examples from event specifications.
This comment contains the modifications that would be made to the copy-pasteable event listeners in documentation, for PR authors and reviewers to inspect for correctness, and will automatically be kept up-to-date whenever this PR is updated.

@robojumper
Copy link
Member Author

Formatting looks good again.

@robojumper robojumper closed this Oct 21, 2023
@Iridar
Copy link
Contributor

Iridar commented Oct 21, 2023

o7

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.

2 participants