Fix grammar and spelling errors across lecture files #688
Conversation
- ar1_processes.md: Fix missing asterisk in math notation - business_cycle.md: Fix preposition usage and grammar - cagan_adaptive.md: Fix subject-verb agreement - long_run_growth.md: Remove redundant word and improve word choice - lp_intro.md: Fix markdown formatting - supply_demand_heterogeneity.md: Fix article usage - tax_smooth.md: Remove duplicate word and extra word - unpleasant.md: Clarify pronoun reference 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
✅ Deploy Preview for taupe-gaufre-c4e660 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| And we can use a theoretical AR(1) model to calculate the right hand side. | ||
|
|
||
| If $\frac{1}{m} \sum_{t = 1}^m X_t$ is not close to $\psi^(x)$, even for many | ||
| If $\frac{1}{m} \sum_{t = 1}^m X_t$ is not close to $\psi^*(x)$, even for many |
There was a problem hiding this comment.
ar1_processes.md: Fixed missing asterisk in math notation ($\psi^*(x)$)
|
thanks @jstac this looks great. Just had one question, then I can merge. |
There was a problem hiding this comment.
Pull request overview
This PR fixes 10 grammar and spelling errors across 8 lecture markdown files, improving readability and correctness of the documentation.
- Corrected grammar issues including article usage, subject-verb agreement, and preposition usage
- Fixed markdown formatting inconsistencies for proper rendering
- Corrected mathematical notation to ensure accurate representation
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| lectures/unpleasant.md | Clarified pronoun reference by adding "that" before "lecture" |
| lectures/tax_smooth.md | Removed duplicate words "use" and "chooses" for better clarity |
| lectures/supply_demand_heterogeneity.md | Fixed article usage from "An" to "A" before "simple example" |
| lectures/lp_intro.md | Corrected markdown italic formatting with proper asterisk placement |
| lectures/long_run_growth.md | Removed redundant "across" and changed "non-continuous" to "incomplete" for accuracy |
| lectures/cagan_adaptive.md | Fixed subject-verb agreement by changing "decline" to "declines" |
| lectures/business_cycle.md | Corrected prepositions from "over" to "in" and "towards" to "in" |
| lectures/ar1_processes.md | Fixed mathematical notation by adding missing asterisk in |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@jstac please feel free to merge when ready. |
|
Thanks @mmcky On my end it says "Merging is blocked. At least 1 approving review is required by reviewers with write access." Does this repo have a different access policy to the other ones? |
|
@jstac I added controls to main at a time when some RAs were still learning and there were some accidental merges. You can click the box to bypass controls. |
|
Thanks @mmcky . That makes sense. But perhaps it's time to unify policies across the repos? |
|
thanks @jstac -- making live now. |
Summary
Files Changed
Test plan
🤖 Generated with Claude Code