Skip to content

Commit

Permalink
remove the version on json
Browse files Browse the repository at this point in the history
  • Loading branch information
xuan-cao-swi committed Aug 28, 2024
1 parent 05b0444 commit 1937838
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 4 deletions.
29 changes: 29 additions & 0 deletions ext/oboe_metal/src/oboe.i
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
%module oboe
%{
#include "oboe_api.h"
#include "oboe_debug.h"
%}
%include "stdint.i"
%include "std_string.i"

%newobject Metadata::copy;
%newobject Metadata::fromString;
%newobject Metadata::createEvent;
%newobject Metadata::makeRandom;

%newobject Context::startTrace;
%newobject Context::createEntry;
%newobject Context::createEvent;
%newobject Context::createExit;
%newobject Context::copy;

%newobject Event::startTrace;
%newobject Event::getMetadata;

%apply int *OUTPUT { int *do_metrics, int *do_sample, int *sample_rate, int *sample_source, int *type, int *auth, int *status};
%apply double *OUTPUT { double *bucket_rate, double *bucket_cap};
%apply std::string *OUTPUT { std::string *status_msg, std::string *auth_msg};
%apply unsigned int& OUTPUT { unsigned int& counter, unsigned int& rate, unsigned int& source };

%include "oboe_debug.h"
%include "oboe_api.h"
5 changes: 1 addition & 4 deletions solarwinds_apm.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,7 @@ Gem::Specification.new do |s|
# OTEL dependencies
s.add_dependency('opentelemetry-instrumentation-all', '>= 0.31.0')
s.add_dependency('opentelemetry-sdk', '>= 1.2.0')

# this still gives a warning, would have to be pinned to a minor version
# but that is not necessary and may restrict other gems
s.add_dependency('json', '~> 2.0')
s.add_dependency('json')

Check notice

Code scanning / Rubocop

Dependencies in the gemspec should be alphabetically sorted. Note

Gemspec/OrderedDependencies: Dependencies should be sorted in an alphabetical order within their section of the gemspec. Dependency json should appear before opentelemetry-sdk.

s.required_ruby_version = '>= 2.7.0'
s.executables = s.files.grep(%r{^bin/}) { |f| File.basename(f) }
Expand Down

0 comments on commit 1937838

Please sign in to comment.