Skip to content

Extract duplicate engagement activities text into reusable child documents - #25

Draft
smasongarrison with Copilot wants to merge 7 commits into
mainfrom
copilot/reduce-syllabus-policy-redundancy
Draft

Extract duplicate engagement activities text into reusable child documents#25
smasongarrison with Copilot wants to merge 7 commits into
mainfrom
copilot/reduce-syllabus-policy-redundancy

Conversation

Copilot AI commented Jan 20, 2026

Copy link
Copy Markdown
Contributor

Engagement activities text was duplicated across 3 course files. Extracted to common child documents using bookdown's child chunk pattern.

Changes

  • Created reusable content files:

    • common/content/engagement_activities.Xmd - Standard engagement activities text
    • common/content/engagement_activities_blended.Xmd - Variant for blended sections
  • Updated course files to reference engagement activities:

    • 0201_stats.Rmd - Uses standard engagement activities
    • 0202_personality.Rmd - Uses blended variant
    • 0204_testing.Rmd - Uses standard engagement activities
  • Fixed existing issues:

    • Corrected broken child reference path in policies.Xmd (./covid.Rmd0106_covid.Xmd) for future use
    • Fixed grammar in extracted content ("allow you engage" → "allow you to engage")

Pattern

Before:

## Engagement Activities
There will be multiple engagement activities...

After:

## Engagement Activities
```{r child = './common/content/engagement_activities.Xmd'}

## Note on Policy Sections

General policies (0101-0105) are already rendered as separate bookdown sections and are not duplicated within course files. This PR only addresses the engagement activities text that was duplicated across course syllabi.

<!-- START COPILOT ORIGINAL PROMPT -->



<details>

<summary>Original prompt</summary>

> 
> ----
> 
> *This section details on the original issue you should resolve*
> 
> <issue_title>Are there ways to reduce the redudancy of the syllabus policies</issue_title>
> <issue_description>Are there ways to reduce the redudancy of the syllabus policies?</issue_description>
> 
> ## Comments on the Issue (you are @copilot in this section)
> 
> <comments>
> </comments>
> 


</details>



<!-- START COPILOT CODING AGENT SUFFIX -->

- Fixes smasongarrison/syllabi#24

<!-- START COPILOT CODING AGENT TIPS -->
---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our [2 minute survey](https://gh.io/copilot-coding-agent-survey).

Copilot AI and others added 2 commits January 20, 2026 16:06
Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>
Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>
@smasongarrison

Copy link
Copy Markdown
Owner

@copilot Why are you adding policies to the class-specific page that are covered in the other pages?

Copilot AI and others added 2 commits January 20, 2026 16:11
Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>
Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>
Copilot AI changed the title [WIP] Propose methods to reduce redundancy in syllabus policies Extract duplicate syllabus policies into reusable child documents Jan 20, 2026
Copilot AI requested a review from smasongarrison January 20, 2026 16:15
Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>

Copilot AI commented Jan 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Why are you adding policies to the class-specific page that are covered in the other pages?

You're absolutely right - I misunderstood the bookdown structure. The general policies (0101-0105) are already rendered as separate sections, so duplicating them in each course file would create redundancy, not reduce it.

I've removed those policy includes from all course files in commit ee8849a. The PR now only addresses the actual redundancy: extracting duplicate engagement activities text into reusable child files.

Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>
Copilot AI changed the title Extract duplicate syllabus policies into reusable child documents Extract duplicate engagement activities text into reusable child documents Jan 20, 2026
@smasongarrison

Copy link
Copy Markdown
Owner

@copilot are there any parts of the policy that are redundant?

Copilot AI commented Jan 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot are there any parts of the policy that are redundant?

Yes! The common/content/policies.Xmd file is redundant. It contains an older, condensed version of the same content that's in the individual policy files (0102_engagement.Rmd, 0103_idea.Rmd, 0104_honesty.Rmd, 0105_communcation.Rmd).

Key findings:

  • policies.Xmd is NOT currently being used anywhere in the bookdown
  • The individual files (0102-0105) are significantly more detailed and up-to-date
  • policies.Xmd is missing newer content like the AI ethics policy from 0104_honesty.Rmd
  • The individual files are the current authoritative versions

Would you like me to remove policies.Xmd since it's redundant and outdated?

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