Skip to content

Commit 3b5eedf

Browse files
committed
Clean up RDoc Pages workflow
1 parent caef2cb commit 3b5eedf

14 files changed

Lines changed: 53 additions & 89 deletions

File tree

.document

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,5 @@
11
README.md
22
CONTRIBUTING.md
33
docs/*.md
4-
docs/rdoc-extension-root.c
5-
lib
6-
ext/rubydex/rubydex.c
7-
ext/rubydex/graph.c
8-
ext/rubydex/declaration.c
9-
ext/rubydex/definition.c
10-
ext/rubydex/document.c
11-
ext/rubydex/reference.c
12-
ext/rubydex/signature.c
13-
ext/rubydex/location.c
14-
ext/rubydex/diagnostic.c
4+
lib/**/*.rb
5+
ext/rubydex/*.c

.github/workflows/pages.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,17 +25,17 @@ jobs:
2525
- name: Set up Ruby
2626
uses: ruby/setup-ruby@89f90524b88a01fe6e0b732220432cc6142926af # v1.313.0
2727
with:
28-
ruby-version: "3.4"
28+
ruby-version: "4.0.0"
2929
bundler-cache: true
3030

3131
- name: Setup Pages
32-
uses: actions/configure-pages@v6
32+
uses: actions/configure-pages@v6.0.0
3333

34-
- name: Build and verify documentation
35-
run: bundle exec rake rdoc:verify
34+
- name: Build documentation
35+
run: bundle exec rake rdoc
3636

3737
- name: Upload artifact
38-
uses: actions/upload-pages-artifact@v5
38+
uses: actions/upload-pages-artifact@v5.0.0
3939

4040
deploy:
4141
environment:
@@ -46,4 +46,4 @@ jobs:
4646
steps:
4747
- name: Deploy to GitHub Pages
4848
id: deployment
49-
uses: actions/deploy-pages@v5
49+
uses: actions/deploy-pages@v5.0.0

.rdoc_options

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,6 @@ op_dir: _site
22
title: Rubydex
33
main_page: README.md
44

5-
exclude:
6-
- AGENTS.md
7-
- CLAUDE.md
8-
- tmp
9-
- rust
10-
115
footer_content:
126
DOCUMENTATION:
137
Home: index.html

Rakefile

Lines changed: 0 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -33,61 +33,6 @@ RDoc::Task.new do |doc|
3333
doc.rdoc_dir = "_site"
3434
end
3535

36-
namespace :rdoc do
37-
desc "Verify the generated RDoc site includes the public Ruby and native extension APIs"
38-
task :verify do
39-
Rake::Task["rdoc"].invoke
40-
41-
html_files = FileList["_site/**/*.html"]
42-
abort "Generated RDoc site is empty" if html_files.empty?
43-
44-
expected_pages = {
45-
"_site/Rubydex/Graph.html" => [
46-
'id="class-rubydex-graph"',
47-
'id="method-i-index_workspace"',
48-
'id="method-i-workspace_paths"',
49-
'id="method-i-index_all"',
50-
'id="method-i-index_source"',
51-
'index_source(uri, source, language_id)',
52-
'id="method-i-keyword"',
53-
'id="method-i-exclude_paths"',
54-
'id="method-i-resolve_constant"',
55-
],
56-
"_site/Rubydex/Declaration.html" => [
57-
'id="class-rubydex-declaration"',
58-
],
59-
"_site/Rubydex/Signature.html" => [
60-
'id="class-rubydex-signature"',
61-
],
62-
"_site/Rubydex/Document.html" => [
63-
'id="class-rubydex-document"',
64-
'id="method-i-uri"',
65-
'id="method-i-definitions"',
66-
],
67-
"_site/Rubydex/Definition.html" => [
68-
'id="class-rubydex-definition"',
69-
'id="method-i-location"',
70-
'id="method-i-comments"',
71-
],
72-
"_site/Rubydex/ResolvedConstantReference.html" => [
73-
'id="class-rubydex-resolvedconstantreference"',
74-
'id="method-i-declaration"',
75-
],
76-
}
77-
78-
missing = expected_pages.flat_map do |file, entries|
79-
unless File.exist?(file)
80-
next ["#{file} was not generated"]
81-
end
82-
83-
content = File.read(file)
84-
entries.reject { |entry| content.include?(entry) }.map { |entry| "#{file}: #{entry}" }
85-
end
86-
87-
abort "Generated RDoc site is missing expected API entries:\n - #{missing.join("\n - ")}" unless missing.empty?
88-
end
89-
end
90-
9136
task :lint do
9237
puts "******** Linting ********\n"
9338
Rake::Task["rubocop"].invoke

docs/rdoc-extension-root.c

Lines changed: 0 additions & 11 deletions
This file was deleted.

ext/rubydex/declaration.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,11 @@
55
#include "rustbindings.h"
66
#include "utils.h"
77

8+
/*
9+
* RDoc parser workaround for https://github.com/ruby/rdoc/issues/1744:
10+
* mRubydex = rb_define_module("Rubydex")
11+
*/
12+
813
VALUE cDeclaration;
914
VALUE cNamespace;
1015
VALUE cClass;

ext/rubydex/definition.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@
88
#include "ruby/internal/scan_args.h"
99
#include "rustbindings.h"
1010

11+
/*
12+
* RDoc parser workaround for https://github.com/ruby/rdoc/issues/1744:
13+
* mRubydex = rb_define_module("Rubydex")
14+
*/
15+
1116
static VALUE mRubydex;
1217
static VALUE cInclude;
1318
static VALUE cPrepend;

ext/rubydex/diagnostic.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
#include "diagnostic.h"
22
#include "rustbindings.h"
33

4+
/*
5+
* RDoc parser workaround for https://github.com/ruby/rdoc/issues/1744:
6+
* mRubydex = rb_define_module("Rubydex")
7+
*/
8+
49
VALUE cDiagnostic;
510

611
void rdxi_initialize_diagnostic(VALUE mRubydex) { cDiagnostic = rb_define_class_under(mRubydex, "Diagnostic", rb_cObject); }

ext/rubydex/document.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,11 @@
44
#include "handle.h"
55
#include "rustbindings.h"
66

7+
/*
8+
* RDoc parser workaround for https://github.com/ruby/rdoc/issues/1744:
9+
* mRubydex = rb_define_module("Rubydex")
10+
*/
11+
712
VALUE cDocument;
813

914
/*

ext/rubydex/graph.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@
88
#include "rustbindings.h"
99
#include "utils.h"
1010

11+
/*
12+
* RDoc parser workaround for https://github.com/ruby/rdoc/issues/1744:
13+
* mRubydex = rb_define_module("Rubydex")
14+
*/
15+
1116
static VALUE cGraph;
1217
static VALUE mRubydex;
1318
static VALUE cKeyword;

0 commit comments

Comments
 (0)