Skip to content

Initial Perl samples - #107

Merged
datalogics-tsmith merged 6 commits into
pdfrest:mainfrom
datalogics-kam:pdfcloud-5204-initial-perl-samples
Sep 5, 2025
Merged

datalogics-tsmith merged 6 commits into
pdfrest:mainfrom
datalogics-kam:pdfcloud-5204-initial-perl-samples

Conversation

@datalogics-kam

@datalogics-kam datalogics-kam commented Sep 3, 2025 •

Copy link
Copy Markdown
Contributor

Please review/merge #106 first

Addresses PDFCLOUD-5204

@datalogics-kam
datalogics-kam force-pushed the pdfcloud-5204-initial-perl-samples branch 2 times, most recently from 240472c to 92c9524 Compare September 4, 2025 15:48
Comment thread Perl/README.md

2) Install dependencies from `cpanfile`
- Standard: `cpanm --installdeps .`
- macOS (Homebrew OpenSSL + HTTPS stack): `make install-macos` or `scripts/setup-macos.sh`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I got a
make: *** No rule to make target 'install-macos'. Stop.
error when trying to run the make command. The scripts one worked though

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It Works For Me™. I don't know what to say except that maybe there was a better make if the script had Xcode install the tools?

Comment thread Perl/AGENTS.md

---

## Audience And Tone (Internal)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think I missed this in the Ruby PR but do we want internal notes in our public facing samples? I don't think they are doing any real harm but are they something that we want?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

These are instructions to Codex (or other coding agents) to make it produce outputs that meet the needs of the project in terms of being for evaluators, etc. The AGENTS.md is checked in because it's actually tuned, and will help do the right thing if we ask for more samples.

I think the (internal) is Codex telling itself to keep that part to itself. I think I told it to consider the audience and tone, and it put it in README, and I said no, that's internal, remember it in AGENTS. A lot of the wording here is the LLM talking to future incarnations of itself.

So I think it's no harm.

- Local ignores for Perl
- Define project structure, coding conventions, and testing process.
- Include setup, development, and sample execution instructions.
- Provide commit and PR guidelines, as well as security best practices.
- Added JSON and Multipart payload examples for generating Markdown and rasterized PDFs.
- Introduced a "Complex Flow" Perl script showcasing file conversion and merging.
- Included `.env.example` for API key configuration and optional EU endpoint setup.
- Updated `README.md` with instructions for setup, dependencies, and running samples.
- Added `cpanfile` for dependency management and setup script for macOS.
- Updated all Perl examples to include consistent headers explaining functionality, setup, usage, and output.
- Simplified and clarified setup instructions for API key and optional EU endpoint configuration.
- Improved usage guidance with tips for handling output and non-2xx responses.
- Added URL for GDPR compliance details across Perl examples, `AGENTS.md`, and `.env.example`.
- Replaced custom `.env` file parsing logic with `Dotenv` library across all Perl examples.
- Updated `cpanfile` to include `Dotenv` as a dependency.
- Adjusted related documentation in `AGENTS.md` to reflect the usage of `Dotenv`.
@datalogics-kam
datalogics-kam force-pushed the pdfcloud-5204-initial-perl-samples branch from 92c9524 to db600ef Compare September 4, 2025 23:53
@datalogics-tsmith
datalogics-tsmith merged commit 4dd1a0d into pdfrest:main Sep 5, 2025
1 check passed
@datalogics-kam
datalogics-kam deleted the pdfcloud-5204-initial-perl-samples branch September 5, 2025 14:53
@datalogics-gwalczak datalogics-gwalczak changed the title PDFCLOUD-5204 Initial Perl samples Initial Perl samples Sep 15, 2025
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