Skip to content

update kpn 2025-08#96

Merged
YoungjunNa merged 7 commits into
adatalab:masterfrom
YoungjunNa:master
Aug 1, 2025
Merged

update kpn 2025-08#96
YoungjunNa merged 7 commits into
adatalab:masterfrom
YoungjunNa:master

Conversation

@YoungjunNa

Copy link
Copy Markdown
Member

No description provided.

@YoungjunNa YoungjunNa requested a review from Copilot August 1, 2025 01:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the KPN (Korean Proven Nias) dataset to the 2025-08 version and includes improvements to documentation and project configuration. The changes focus on updating data sources, fixing documentation language, and adding CI/CD workflows.

Key changes:

  • Updates KPN dataset source file from 2502 to 2508 version with adjusted skip parameter
  • Fixes grammatical errors in documentation for the hanwoo_info function
  • Adds GitHub Actions workflow for R package checking and CI/CD automation

Reviewed Changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
data-raw/hanwoo_kpn/DATASET.R Updates Excel data source to newer version (kpn-nias-2508.xlsx) and changes skip parameter
man/hanwoo_info.Rd Corrects grammar in function documentation and parameter descriptions
.github/workflows/check-standard.yaml Adds comprehensive R-CMD-check workflow for CI/CD
.github/.gitignore Adds gitignore for HTML files in GitHub directory
.Rbuildignore Excludes GitHub directory from R package builds

Comment thread data-raw/hanwoo_kpn/DATASET.R
- Remove outdated r.yml workflow
- Keep modern check-standard.yaml workflow
@YoungjunNa YoungjunNa merged commit 8ff90a1 into adatalab:master Aug 1, 2025
0 of 5 checks passed
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